Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

docs: Update README #621

Merged
merged 2 commits into from
May 30, 2024
Merged

docs: Update README #621

merged 2 commits into from
May 30, 2024

Conversation

sarina
Copy link
Contributor

@sarina sarina commented May 29, 2024 •
edited
Loading

  • Remove outdated Travis CI badge
  • s/edX/Open edX/
  • Add note about Tutor - currently unknown if this package works on Tutor, but Devstack is deprecated.
  • Drop Codecov from 100 -> 90 (currently this check has been failing on master for months. Also 80 is the generally accepted standard beyond which you see diminishing results. But - this repo is at 98 so didn't want to drop drastically)

@sarina
docs: Update README
bd0ff58 
* Remove outdated Travis CI badge
* s/edX/Open edX/
* Add note about Tutor - currently unknown if this package works on Tutor, but Devstack is deprecated.
@sarina
Copy link
Contributor Author

sarina commented May 29, 2024

@feanil any objection to knocking down codecov a bit? master is failing on it for quite some time and it seems at this point an ignored check. https://github.com/openedx/registrar/blob/master/codecov.yml

@sarina sarina mentioned this pull request May 30, 2024
Closed
@sarina sarina merged commit 007d55d into master May 30, 2024
6 checks passed
@sarina sarina deleted the sarina-patch-1 branch May 30, 2024 18:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@feanil feanil feanil approved these changes

@davidjoy davidjoy davidjoy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sarina @davidjoy @feanil