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

Republish Pythagorean Triplet because of missing instructions. #414

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

SimaDovakin
Copy link

No description provided.

@BNAndras
Copy link
Member

@ErikSchierboom, if I recall correctly, this type of change affects the Exercism database and some manual work needs to be done to migrate the old slug to the new one. Is that correct?

@SimaDovakin
Copy link
Author

I renamed directory name but now all test fails.

@SimaDovakin
Copy link
Author

I got it. I guess, I should change modules' names, too

@SimaDovakin SimaDovakin requested a review from iHiD February 16, 2025 18:01
@ErikSchierboom
Copy link
Member

Updating just the name is fine

@BNAndras
Copy link
Member

Mind adding yourself inside the contributors array to the exercise config as well? See https://exercism.org/docs/building/tracks/practice-exercises#h-file-meta-config-json for details.

@SimaDovakin
Copy link
Author

Mind adding yourself inside the contributors array to the exercise config as well? See https://exercism.org/docs/building/tracks/practice-exercises#h-file-meta-config-json for details.

Forgot about it, too)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants