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

Temporarily revert breaking changes for bugfix release #1224

Merged
merged 5 commits into from
Dec 20, 2024
Merged

Temporarily revert breaking changes for bugfix release #1224

merged 5 commits into from
Dec 20, 2024

Conversation

rly
Copy link
Contributor

@rly rly commented Dec 20, 2024

We had been preparing a major 4.0 release, but I think it could be a major burden on downstream tools to present breaking changes right before the winter holidays. However, there are some bug fixes that we would like to release in the meantime. I propose we revert the breaking changes (this PR), make a bugfix release, and add the breaking changes back in to dev. I can then make a pre-release and aim to do the major release on/around Jan 2.

The commit history will be a bit ugly, but I think that's OK. (We could adopt special release branches to make this situation easier in the future but that complicates the workflow and I think this situation is not that common.)

@rly rly marked this pull request as ready for review December 20, 2024 18:14
@rly rly requested a review from oruebel December 20, 2024 18:14
@rly rly enabled auto-merge (squash) December 20, 2024 18:15
@rly rly merged commit 035d3ec into dev Dec 20, 2024
27 checks passed
@rly rly deleted the temp_revert branch December 20, 2024 18:18
@rly rly mentioned this pull request Dec 20, 2024
13 tasks
Copy link

codecov bot commented Dec 20, 2024

Codecov Report

Attention: Patch coverage is 53.02419% with 233 lines in your changes missing coverage. Please review.

Project coverage is 89.00%. Comparing base (6396a81) to head (5f708f2).
Report is 1 commits behind head on dev.

Files with missing lines Patch % Lines
src/hdmf/array.py 52.55% 57 Missing and 8 partials ⚠️
src/hdmf/backends/hdf5/h5tools.py 30.43% 41 Missing and 7 partials ⚠️
src/hdmf/query.py 51.80% 30 Missing and 10 partials ⚠️
src/hdmf/region.py 47.72% 23 Missing ⚠️
src/hdmf/backends/hdf5/h5_utils.py 53.19% 21 Missing and 1 partial ⚠️
src/hdmf/build/objectmapper.py 53.57% 12 Missing and 1 partial ⚠️
src/hdmf/__init__.py 38.46% 8 Missing ⚠️
src/hdmf/build/map.py 0.00% 4 Missing ⚠️
src/hdmf/container.py 81.25% 1 Missing and 2 partials ⚠️
src/hdmf/build/manager.py 50.00% 2 Missing ⚠️
... and 3 more
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #1224      +/-   ##
==========================================
- Coverage   90.86%   89.00%   -1.86%     
==========================================
  Files          42       45       +3     
  Lines        9520     9965     +445     
  Branches     2723     2831     +108     
==========================================
+ Hits         8650     8869     +219     
- Misses        576      774     +198     
- Partials      294      322      +28     

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants