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

NP-92 Create diffing script for degrees #678

Merged
merged 4 commits into from
Sep 26, 2023
Merged

NP-92 Create diffing script for degrees #678

merged 4 commits into from
Sep 26, 2023

Conversation

Peytonbarre
Copy link
Contributor

Overview

Adds NP-92

To solve for major requirements changing each year, this diffing script will through each major and raise a JIRA ticket if there are any changes. As of now, the script detects if there are any changes in courses (renamed/added/deleted) or if a major/concentration was removed. This script is ran annually before the fall semester and also has a manual github action dispatch.

What Changed

Created a github workflow that has a manual dispatch and cron scheduler to run the diffing script (planner/validator/scripts/diff.py).

Other Notes

After the versioning system is complete, I will change the file parsing (for the case when majors are removed to ensure they don't parse and raise a ticket every year). After versioning system I will also change the script to support core curriculum changes.

Let me know if you need more information/documentation!

@Peytonbarre Peytonbarre requested review from a team and akevinge as code owners September 23, 2023 06:28
@Peytonbarre Peytonbarre changed the title diff script NP-92 Create diffing script for degrees Sep 23, 2023
@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

Copy link
Member

@akevinge akevinge left a comment

Choose a reason for hiding this comment

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

Small nit, other than that it should work!

.github/workflows/versioning.yml Show resolved Hide resolved
@github-actions
Copy link

@akevinge akevinge merged commit c0fb5be into develop Sep 26, 2023
6 checks passed
@akevinge akevinge deleted the feat/diff branch September 26, 2023 19:50
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.

2 participants