-
Notifications
You must be signed in to change notification settings - Fork 544
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
Update Github action workflow #79
base: full-template
Are you sure you want to change the base?
Update Github action workflow #79
Conversation
@NereusWB922 Thanks for this PR. Is this an urgent change or can we delay it till next semester? This branch will be put into use very soon (when CS2113 tP starts, so we need to decide now) |
I don't think it's urgent since the current version isn't causing any issues. We can delay it until next semester without any problems. |
@se-edu/tech-team-level1 For your review ... |
@NereusWB922 Looks like this PR got messed up after the target branch was rebased. Perhaps close this PR and PR again? |
e3fb8a7
to
31aff38
Compare
@damithc Hi Prof, just rebased this branch onto the updated target branch. |
|
||
- name: Merge to master | ||
run: git checkout --progress --force ${{ github.sha }} | ||
uses: actions/checkout@v4 |
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.
@NereusWB922 does the behavior remains the same here?
Fixes #66
Bumps version:
Changes made: