-
Notifications
You must be signed in to change notification settings - Fork 375
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
Problem authorship #2303
base: master
Are you sure you want to change the base?
Problem authorship #2303
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2303 +/- ##
==========================================
+ Coverage 46.76% 46.83% +0.06%
==========================================
Files 251 253 +2
Lines 13317 13360 +43
==========================================
+ Hits 6228 6257 +29
- Misses 7089 7103 +14 ☔ View full report in Codecov by Sentry. |
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.
Just some minor nits, otherwise LGTM, thanks!
be25e61
to
d38bc14
Compare
Ready to merge, changes from #2304 have been integrated :) |
c35f69b
to
c6cbea5
Compare
… will be displayed at the end of the problem's statement.
663d217
to
341d70c
Compare
Hi there! I just integrated your "master" last changes into this branch, and also I finally learned how to squash commits properly (sorry for the delay), so ready to rock'n'roll 😊 I'll keep the branch unmodified till the PR acceptance, but feel free to ask or request as you need if you need further modifications. Kind regards! |
Related with: #2298
Related also with: #144
A new option has been included for attribution purposes, in order to give the authorship to external DMOJ users (like websites, books, etc):
This data will be displayed at the end of the problem statement, just before the license:
Thanks and have a nice Xmas!
Fher