-
Notifications
You must be signed in to change notification settings - Fork 25
Conversation
@navinkarkera pinging you on this PR, as there appeared to be several other locations that claim some form of ownership on the repo. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #418 +/- ##
=======================================
Coverage 53.14% 53.14%
=======================================
Files 80 80
Lines 2021 2021
Branches 368 368
=======================================
Hits 1074 1074
Misses 914 914
Partials 33 33 ☔ View full report in Codecov by Sentry. |
@jristau1984 Thanks! Do you thinks it makes sense to add my name in these files as well? cc: @feanil |
I tagged you instead of replacing our team with your name because I had that exact same question. I don't know what each and every one of these files is really supposed to do, so I think @feanil could offer better guidance. |
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.
although it is ok that owner is not specified, per:
https://open-edx-proposals.readthedocs.io/en/latest/archived/oep-0002-bp-repo-metadata.html#id4
An empty space is not valid YAML, and this will fail reporting tools. Instead, please add '~' or 'null' or ''''.
Co-authored-by: connorhaugh <49422820+connorhaugh@users.noreply.github.com>
Continuing maintainership updates for 2U 2024.