-
-
Notifications
You must be signed in to change notification settings - Fork 242
Manchester | 25-ITP-Sep | Mahtem T. Mengstu | Sprint 3 | coursework/sprint-3-stretch #884
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
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.
Implementation and testing
This does not quite describe what was changed in this PR branch (compared to CYF's main). Can you make the description more informative?
Are the changes made in "2-practice-tdd" related to this PR?
Thank you @cjyuan, because I created a branch from the previous branch the commits of the previous branch "2-practice-tdd" were mixed with this PR. Now, I have created a new fixed branch (coursework/sprint-3-stretch-Fixed) with only related commits and files. New and fixed PR: Manchester | 25-ITP-Sep | Mahtem T. Mengstu | Sprint 3 | coursework/sprint-3-stretch-Fixed #884 #889 |
|
You could practice using git command to rebase this branch onto |
|
Since you have created a separate PR for Sprint-3/stretch, you may want to close this branch. |
Okay, I will close it. |
Learners, PR Template
Self checklist
Changelist
Implementation and testing
Questions
Questions will be forwarded on time.