Skip to content

Fix path for unit tests badge#61

Closed
rlskoeser wants to merge 4 commits into
developfrom
rlskoeser-patch-1
Closed

Fix path for unit tests badge#61
rlskoeser wants to merge 4 commits into
developfrom
rlskoeser-patch-1

Conversation

@rlskoeser
Copy link
Copy Markdown
Collaborator

I just noticed the unit tests badge wasn't displaying on the main readme. I don't think it's worth a patch release - is there any way to update main without that? Should we just pull into develop for next week?

rlskoeser and others added 4 commits October 28, 2025 10:06
* Update version number

* Update software release checklist

* Add issue template to changelog

* Remove redundant gitflow config option
@rlskoeser rlskoeser requested a review from laurejt May 20, 2026 14:22
@codecov
Copy link
Copy Markdown

codecov Bot commented May 20, 2026

Codecov Report

❌ Patch coverage is 61.71096% with 461 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.07%. Comparing base (6e7dc21) to head (f6f026d).
⚠️ Report is 59 commits behind head on develop.

❌ Your patch status has failed because the patch coverage (61.71%) is below the target coverage (95.00%). You can increase the patch coverage or adjust the target coverage.

❗ There is a different number of reports uploaded between BASE (6e7dc21) and HEAD (f6f026d). Click for more details.

HEAD has 29 uploads less than BASE
Flag BASE (6e7dc21) HEAD (f6f026d)
32 3
Additional details and impacted files
@@             Coverage Diff              @@
##           develop      #61       +/-   ##
============================================
- Coverage    99.71%   76.07%   -23.65%     
============================================
  Files            3       16       +13     
  Lines          354     1960     +1606     
  Branches         0      158      +158     
============================================
+ Hits           353     1491     +1138     
- Misses           1      436      +435     
- Partials         0       33       +33     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@laurejt
Copy link
Copy Markdown
Contributor

laurejt commented May 20, 2026

I just noticed the unit tests badge wasn't displaying on the main readme. I don't think it's worth a patch release - is there any way to update main without that? Should we just pull into develop for next week?

I did notice this, sorry if I forgot to mention it. I figured I'd wrap it into a bigger feature branch that fixes all the small issues I caught while preparing the release for viapy.

@laurejt laurejt added the no changelog Skip changelog requirement for non-user-visible changes label May 20, 2026
laurejt
laurejt previously approved these changes May 20, 2026
Copy link
Copy Markdown
Contributor

@laurejt laurejt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@rlskoeser rlskoeser changed the base branch from main to develop May 20, 2026 20:19
@rlskoeser rlskoeser dismissed laurejt’s stale review May 20, 2026 20:19

The base branch was changed.

@rlskoeser rlskoeser closed this May 26, 2026
@rlskoeser rlskoeser deleted the rlskoeser-patch-1 branch May 26, 2026 21:45
@rlskoeser
Copy link
Copy Markdown
Collaborator Author

closing in favor of #63

@laurejt laurejt restored the rlskoeser-patch-1 branch May 27, 2026 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog Skip changelog requirement for non-user-visible changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants