-
Notifications
You must be signed in to change notification settings - Fork 27
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
DOC Add doc in CommandRegistry
about when it is updated
#316
Conversation
I don't think the CI failures are related to this doc change. |
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.
Small nits, thanks @lucyleeow 🙏
Looks like our codecov token might have expired, or something: [2023-10-10T02:14:55.728Z] ['error'] There was an error running the uploader: Error uploading to [https://codecov.io:]
(https://codecov.io/) Error: There was an error fetching the storage URL during POST: 400 - [ErrorDetail(string='This
repository has been deactivated. To resume uploading to it, please activate the repository in the codecov UI:
https://app.codecov.io/github/napari/npe2/settings', code='invalid')]
@Czaki do you know what I'd need to do to fix this? |
Co-authored-by: Draga Doncila Pop <[email protected]>
Co-authored-by: Draga Doncila Pop <[email protected]>
Thanks changes made! |
Codecov Report
@@ Coverage Diff @@
## main #316 +/- ##
=======================================
Coverage 99.75% 99.75%
=======================================
Files 37 37
Lines 2814 2814
=======================================
Hits 2807 2807
Misses 7 7
|
Adds some info about when
CommandRegistry
is updated. Uses info learned when debugging #314Happy to change wording or move to a better place.