-
Notifications
You must be signed in to change notification settings - Fork 9
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
learn: Add course catalog exporting #378
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
juliaogris
force-pushed
the
learn-catalog
branch
from
June 12, 2024 02:42
0f0ffa3
to
3ba74f8
Compare
juliaogris
force-pushed
the
learn-catalog
branch
from
June 12, 2024 03:21
3ba74f8
to
783ec8f
Compare
juliaogris
force-pushed
the
learn-catalog
branch
from
June 12, 2024 03:24
090ca0a
to
34dd5a2
Compare
juliaogris
force-pushed
the
learn-catalog
branch
from
June 12, 2024 03:28
34dd5a2
to
954a40c
Compare
juliaogris
force-pushed
the
learn-unit
branch
3 times, most recently
from
June 13, 2024 06:10
df52993
to
9a3340e
Compare
Fix up CLI docs in learn/README.md as they have gotten out of sync with recent refactorings. oops.
Upgrade go dependencies with cd learn go get -u ./... and remove replace directive as rsc.io/markdown has been patched.
Add Name field to Course, Unit and Exercise, which is extracted from the first heading of the Markdown document with styling stripped. This text will be used in a follow-up commit for the catalog.
juliaogris
force-pushed
the
learn-catalog
branch
from
June 13, 2024 06:37
954a40c
to
c0de0db
Compare
camh-
approved these changes
Jun 13, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
☘️ LGTM
Add course catalog exporting as single JSON file. Catalog contains all information to generate a question set for an exercise, quiz or unittest identified by composite IDs. It also holds the maximumScore achievable for each course and unit. An exercise has a predefined score of 10. This will allow for progress tracking as percentage.
juliaogris
force-pushed
the
learn-catalog
branch
from
June 13, 2024 12:30
c0de0db
to
eba2d1b
Compare
30 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Add course catalog exporting as single JSON file. Catalog contains all
information to generate a question set for an exercise, quiz or unittest
identified by composite IDs. It also holds the maximumScore achievable for
each course and unit. An exercise has a predefined score of 10. This will
allow for progress tracking as percentage.
Sneak in a couple of preparatory and clean-up commits: fix learn/README.md
update go dependencies and add Name field to some markdown model types.