-
Notifications
You must be signed in to change notification settings - Fork 11
A2 mistakes
shinramenisbae edited this page Oct 9, 2024
·
2 revisions
For some of the pull requests, our group performed the necessary code reviews and commenting. However, we made mistakes in choose the merging method as we did not use the squash and merge option.
We have now realised this mistake and are working on contributing towards the project strictly following the codes of conduct and good git etiquttes.
It took us a while to set up sonarcloud and snyk after the assignment handover because we decided to create a new github organization just for this assignment. This led to some difficulty for snyk and sonarcloud detecting the newly transferred repository. However, we have both of them set up now and all the pull requests are being automatically checked with them.