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

ADD: Beta status badge #210

Closed
wants to merge 11 commits into from

Conversation

syedhamidali
Copy link
Contributor

  • Closes #xxxx
  • Tests added
  • Changes are documented in history.md

@syedhamidali syedhamidali changed the title Syedhamidali patch 1 ADD: Beta status badge Sep 18, 2024
Copy link

codecov bot commented Sep 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.97%. Comparing base (a827a18) to head (69104bd).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #210      +/-   ##
==========================================
+ Coverage   91.95%   91.97%   +0.02%     
==========================================
  Files          23       23              
  Lines        4562     4574      +12     
==========================================
+ Hits         4195     4207      +12     
  Misses        367      367              
Flag Coverage Δ
notebooktests 78.15% <30.76%> (-0.14%) ⬇️
unittests 90.24% <100.00%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kmuehlbauer
Copy link
Collaborator

@syedhamidali As commented inline, please remove all additional code changes from the beta-badge PR, then we can merge.

Side note: On the openradar dev meeting before the ERAD, the was the conclusion to move fast to approach version 1.0.

@syedhamidali syedhamidali deleted the syedhamidali-patch-1 branch September 19, 2024 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants