Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate scale-generator #585

Closed
wants to merge 3 commits into from
Closed

Deprecate scale-generator #585

wants to merge 3 commits into from

Conversation

BNAndras
Copy link
Member

@neenjaw
Copy link
Collaborator

neenjaw commented Nov 27, 2023

@BNAndras I'm not sure what to do with this yet. I can see that it can be a hard toy problem for some students, and it isn't in the best shape, but I also think that to deprecate means invalidating a lot of solutions that have been done for this problem.

@neenjaw neenjaw added x:size/tiny Tiny amount of work x:rep/tiny Tiny amount of reputation labels Nov 27, 2023
@neenjaw neenjaw self-assigned this Nov 27, 2023
@neenjaw neenjaw added the paused Work paused until further notice label Nov 27, 2023
@mk-mxp
Copy link
Contributor

mk-mxp commented Apr 21, 2024

As shown in the forum and described in the documentation, there is no invalidation of solutions (if that means they are lost for the students or students loose reputation earned). The exercise will be hidden for new students only.

If there is no other reason not to deprecate this exercise, I suggest: Let's get that merged!

@mk-mxp
Copy link
Contributor

mk-mxp commented May 7, 2024

@BNAndras I can't rebase this branch to fix the formatting issues. Do you want to rebase your branch or should I close this and deprecate the exercise in a new PR?

@mk-mxp mk-mxp removed the paused Work paused until further notice label May 19, 2024
mk-mxp added a commit to mk-mxp/php that referenced this pull request May 19, 2024
@mk-mxp mk-mxp mentioned this pull request May 19, 2024
@mk-mxp mk-mxp closed this May 19, 2024
mk-mxp added a commit to mk-mxp/php that referenced this pull request May 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
x:rep/tiny Tiny amount of reputation x:size/tiny Tiny amount of work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants