-
Notifications
You must be signed in to change notification settings - Fork 2
Adding the script for adding and removing problems from the any course #17
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
Conversation
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.
Pull Request Overview
This PR implements functionality to add and remove problems from course assignments via JSON configuration and automation. The changes enable automated management of OmegaUp course content through a JSON file that specifies which problems to add or remove from specific assignments.
Key changes include:
- Creation of a Python script for automated problem addition/removal from course assignments
- Addition of a JSON configuration file for specifying problem operations
- Implementation of GitHub Actions workflow for automated execution
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
File | Description |
---|---|
utils/update_assignment_problems.py |
Main script implementing problem addition/removal logic with API integration and file management |
adding_removing_problems.json |
Configuration file with empty arrays for specifying problems to add/remove |
.github/workflows/modify-assignments.yaml |
GitHub Actions workflow to automatically run the problem update script |
Comments suppressed due to low confidence (2)
.github/workflows/modify-assignments.yaml:32
- The pipenv version 2023.11.15 may not exist. Consider using a known stable version or removing the version constraint to use the latest available version.
pip install pipenv==2023.11.15
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.
In addition to Copilot’s suggestions, I left you some extra comments.
- removing global var and using dep injection instead
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.
Great!
Fixes: #16
adding_removing_problems.json
file which would contain the list of problems, having alias of assignment and course in which the problem needs to be added or removed.update_assignment_problems.py
script which fetched the problems to be added or removed from theadding_removing_problems.json
file and has functions-adding_removing_problems.json
.modify-assignments.yaml
which would run theupdate_assignment_problems.py
for automatic integration.