AY2425S1-CS2113-T10-3 / tp

https://ay2425s1-cs2113-t10-3.github.io/tp/
0 stars 4 forks source link

Update code sections authors for RepoSense #48

Closed xenthm closed 2 days ago

xenthm commented 2 days ago

Based on Week 10 tasks, //@@author [github user] should be used sparingly because its effects extend until another //@@author [github user] annotation is found or the end of the file is reached.

Moving forward, we should strive to "close" our //@@author [github user] annotations such that we don't inadvertently claim more code than intended.

Also, a lesson learnt is that syncing a updated upstream master with a local development branch (with conflicts there were resolved) messes with the native GitHub log system and incorrectly attributes all merged code as authored by the developer who synced. Try to limit this for now.