-
-
Notifications
You must be signed in to change notification settings - Fork 273
Manchester | 25-ITP-Sep| Fithi-Teklom| Sprint 2 | Coursework #827
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
Manchester | 25-ITP-Sep| Fithi-Teklom| Sprint 2 | Coursework #827
Conversation
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Sprint part (Sprint2) doesn't match expected format (example: 'Sprint 2', without quotes) If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
1 similar comment
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Sprint part (Sprint2) doesn't match expected format (example: 'Sprint 2', without quotes) If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Sprint part (Sprint2) doesn't match expected format (example: 'Sprint 2', without quotes) If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Sprint part (Sprint2) doesn't match expected format (example: 'Sprint 2', without quotes) If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
cjyuan
left a comment
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.
-
Can you revert the changes made to all the files that's not related to Sprint-2 exercise to keep this PR branch clean?
-
The PR description is missing a brief description of this PR. Can you update the PR title and the PR description according to this PR Guide?
|
Reverting changes means undoing the changes (and not deleting the modified files). One way to ensure the files in Sprint-1 folder are identical to the original version of the same files is to replace the Sprint-1 folder by the version in your
|
|
So what am I going to do with the files I have deleted? Is there any
solution for it?
…On Thu, 6 Nov 2025, 20:01 CJ Yuan, ***@***.***> wrote:
*cjyuan* left a comment
(CodeYourFuture/Module-Structuring-and-Testing-Data#827)
<#827 (comment)>
The PR is missing a brief description and the last component in the title
should be just "Coursework"
image.png (view on web)
<https://github.com/user-attachments/assets/d415b8e0-68b3-4419-85ca-f31d8bd8402d>
—
Reply to this email directly, view it on GitHub
<#827 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BTYM74VF4OO6YZKFDKN636T33OSKDAVCNFSM6AAAAACKVN5ALGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTIOJZGE3TQMRYGM>
.
You are receiving this because you authored the thread.Message ID:
<CodeYourFuture/Module-Structuring-and-Testing-Data/pull/827/c3499178283@
github.com>
|
|
Each commit represents a snapshot of the repo. If the files appear in a past commit, then there is a way to recover them. In your repo, since the original files are also available in the |
|
So does this mean, I have to redo it again?
…On Thu, 6 Nov 2025, 20:22 CJ Yuan, ***@***.***> wrote:
*cjyuan* left a comment
(CodeYourFuture/Module-Structuring-and-Testing-Data#827)
<#827 (comment)>
Each commit represents a snapshot of the repo. If the files appear in a
past commit, then there is a way to recover them.
In your repo, since the original files are also available in the main
branch, it might be easier just to copy the files from your main branch
(instead of from a past commit)
—
Reply to this email directly, view it on GitHub
<#827 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BTYM74QNFOWXD5RLVHIZTHD33OUXXAVCNFSM6AAAAACKVN5ALGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTIOJZGI2DEMJTHA>
.
You are receiving this because you authored the thread.Message ID:
<CodeYourFuture/Module-Structuring-and-Testing-Data/pull/827/c3499242138@
github.com>
|
|
You can seek help from a volunteer on Saturday to fix this branch, or you can close this PR and then create another PR. |

Self checklist