nus-cs2103-AY2223S2 / forum

12 stars 0 forks source link

Help: As per the tP progress dashboard, you have not yet merged any PRs that follow the strict forking workflow (i.e., in the past two iterations and the current iteration so far). #270

Open sumhungyee opened 1 year ago

sumhungyee commented 1 year ago

I accidentally merged my master branch instead of creating new branches to be merged, and I think as a result these are not detected by the system, even though I have already tagged the milestones. Can I have some help? I think v1.2 milestones have closed as well.

damithc commented 1 year ago

@sumhungyee It's fine if you follow the correct workflow for future PRs. The previous items in concern will remain red, but it's fine. Having a few red items doesn't affect marks, unless you have many such items. The important thing is that you learn the correct forking workflow at the end.

sumhungyee commented 1 year ago

thanks a lot prof