Fix Sparta Sync command and allow for local testing #6
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR has two main goals:
sparta sync
overrides the repository even when the exercises already exists with the same SHA,sparta init
has now two hidden flags:--force
(or-f
) and--spartaURL="url"
.Both have default values so it will not change the usage for students.
--spartaURL
is there to change the URL that the CLI will ask for the calendar (which can now be set to our.local
instance) and will record that in theconfig.json
that the CLI generates at init--force
is a convenience option to recreate anexercises
directory when callingsparta init
.Motivation and Context
We couldn't tell the previous students to use
sparta sync
even if they missed some day because it would have overwritten all their exercises.How Has This Been Tested?
Manually locally
Types of changes
Chore (non-breaking change which refactors / improves the existing code base)New feature (non-breaking change which adds functionality)Breaking change (fix or feature that would cause existing functionality tochange)
Checklist: