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

Add prerequisites for practice exercises #1030

Merged

Conversation

TomPradat
Copy link
Contributor

@TomPradat TomPradat commented Mar 2, 2021

This adresses the issue #982

What's left :

  • simple-cipher
  • wordy
  • secret-handshake

config.json Outdated Show resolved Hide resolved
config.json Outdated Show resolved Hide resolved
config.json Outdated
@@ -168,7 +168,7 @@
"name": "Resistor Color Duo",
"uuid": "de800041-3dcc-41b9-b101-7314ff685c93",
"practices": [],
"prerequisites": [],
"prerequisites": ["arrays"],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe better: array-transformations?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you explain what you mean by "array-transformations" and "array-analysis", I don't get the subtlety

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Got it 👌

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel like this would be better to put "array-analysis" just like in "Resistor Color". IMO this could also be a good exercise to add "array-destructuring", what do you think ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good @TomPradat

config.json Outdated Show resolved Hide resolved
config.json Outdated Show resolved Hide resolved
config.json Outdated Show resolved Hide resolved
config.json Outdated Show resolved Hide resolved
config.json Outdated Show resolved Hide resolved
config.json Show resolved Hide resolved
config.json Outdated Show resolved Hide resolved
@TomPradat TomPradat force-pushed the feature/practice-exercises-prerequisites branch from ea14608 to d7d7a89 Compare March 28, 2021 20:39
@TomPradat TomPradat force-pushed the feature/practice-exercises-prerequisites branch from d7d7a89 to 525b035 Compare April 1, 2021 20:19
@SleeplessByte
Copy link
Member

@TomPradat let me know if this is ready. You can hit Squash and merge yourself if ya want.

@TomPradat
Copy link
Contributor Author

@TomPradat let me know if this is ready. You can hit Squash and merge yourself if ya want.

I meant to check everything one last time this week, I'll let you know when this is good for me :)

@TomPradat
Copy link
Contributor Author

@SleeplessByte I'm ready for the merge if you are :)

@TomPradat TomPradat changed the title WIP : Add prerequisites for practice exercises Add prerequisites for practice exercises Apr 8, 2021
config.json Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore 🔧 Meta related task such as build, test, linting, maintainers.json etc. v3-migration 🤖 Preparing for Exercism v3 x:size/large Large amount of work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants