Skip to content
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

Collated Codes #198

Merged
merged 4 commits into from
Nov 7, 2016
Merged

Collated Codes #198

merged 4 commits into from
Nov 7, 2016

Conversation

ChaseTiong
Copy link

This is my collated codes for submission for V0.5. Please do help me delete the old one if you guys do collate and found any.

Or if you guys find any codes that is written by you guys, please inform me so I can make the changes. However, do let me know earlier like anytime before 10pm.

@louietyj
Copy link

louietyj commented Nov 7, 2016

Hi Yaocong, please check through your unused collates. We have found a lot of code in there that are trivial reuses of existing code by us. For example, all your fixtures were directly plagiarized from us with insignificant modifications of names and dates. The entire structure is so similar that it looks copy-pasted.

In the spirit of DRY, within an internal project we always encourage code reuse and we have been quite generous about claiming reused with modifications code. But when you decide to fork from our repo, it is not fair to continue reusing our code and claiming it as "unused" when it is clear that these are about 90% derivative of our existing work.

Irvin and I think that it would be fair for you to mark these chunks as "reused".

@ChaseTiong
Copy link
Author

@louietyj I have recollate unused files with the variables to be reuse

@ChaseTiong
Copy link
Author

ChaseTiong commented Nov 7, 2016

Let me know if there are any part that require changes. Thank you

@irvinlim irvinlim merged commit a418571 into master Nov 7, 2016
@irvinlim
Copy link

irvinlim commented Nov 7, 2016

Will be taking the unused/reused files, the others will be recollated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants