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

Reorder kata directories #186

Closed

Conversation

adamatan
Copy link
Contributor

@adamatan adamatan commented Jul 4, 2019

Fixes #185 .

This PR is meant to be a basis for discussion following the conversation on #185. The rationale behind the numbering is explained in the commit message.

If and when the order is approved, we might add next/prev links in the README files.

This commit rearranges the Katas to create a clear learning path for the basic concepts, and group advanced topics together.

0xxx - Basics, from staging to interactive rebase
1xxx - Commits
2xxx - Merges, branches, objects
3xxx - Configuration
4xxx - Other advanced topics

Other changes:
1. Removed the "basic" prefix
2. Renamed "advanced-rebase-interactive" to the simpler "interactive rebase"
@adamatan adamatan mentioned this pull request Jul 5, 2019
@JKrag
Copy link
Contributor

JKrag commented Jul 9, 2019

I would have liked to separate the discussion of numeric prefixes from any talk of renaming the actual exercises.
I have commented on the numeric prefixes in the original issue #185 .
The renaming of exercises is an open discussion in #156 and also a more specific changes in #172

Finally a small comment on the renaming of the interactive rebase kata. I called it advanced because it is a very unguided free-form exercise, and I was planning on creating 1 or two more introductory exercises as well. Whether the "advanced" part should be moved to a postfix instead is another discussion (aka #156 again).

@RandomSort
Copy link
Contributor

Hi @adamatan
Thank you for your contribution. I will reject and close this PR as the fix ended up being different: #189

@RandomSort RandomSort closed this Aug 5, 2019
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.

Ordering the katas
3 participants