Skip to content

Conversation

@rozyczko
Copy link
Member

@rozyczko rozyczko commented Dec 2, 2025

The update to the doc workflow could have waited but the link to the documentation site should really be fixed.

@rozyczko rozyczko added [scope] maintenance Code/tooling cleanup, no feature or bugfix (major.minor.PATCH) [priority] highest Urgent. Needs attention ASAP labels Dec 2, 2025
@codecov
Copy link

codecov bot commented Dec 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.80%. Comparing base (366e324) to head (9ff2bcd).
⚠️ Report is 15 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #171   +/-   ##
=======================================
  Coverage   80.80%   80.80%           
=======================================
  Files          52       52           
  Lines        4261     4261           
  Branches      739      739           
=======================================
  Hits         3443     3443           
  Misses        633      633           
  Partials      185      185           
Flag Coverage Δ
unittests 80.80% <ø> (ø)

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

Copy link
Contributor

@damskii9992 damskii9992 left a comment

Choose a reason for hiding this comment

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

This should go into develop first. We can just make the PR into master immediately after.
We probably don't even have to make a new release on master.

@rozyczko
Copy link
Member Author

rozyczko commented Dec 3, 2025

This should go into develop first. We can just make the PR into master immediately after. We probably don't even have to make a new release on master.

Nope. From https://github.com/orgs/easyscience/discussions/12

Hotfix branches are small fixes going directly off/to main with release fixes. Those need to also be merged to develop immediately.

@rozyczko rozyczko merged commit 0fd4c37 into master Dec 3, 2025
41 checks passed
@damskii9992
Copy link
Contributor

I just realized, this hotfix should probably have been a squash merge . . .

@damskii9992 damskii9992 mentioned this pull request Dec 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[priority] highest Urgent. Needs attention ASAP [scope] maintenance Code/tooling cleanup, no feature or bugfix (major.minor.PATCH)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants