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

Release v0.24.0 #683

Merged
merged 30 commits into from
Oct 3, 2024
Merged

Release v0.24.0 #683

merged 30 commits into from
Oct 3, 2024

Conversation

gammazero
Copy link
Contributor

Closes #680

Required by kubo for Upgrade to Boxo v0.24.0

lidel and others added 28 commits September 5, 2024 23:45
* ipld/unixfs/hamt: catch panic in walkChildren

* Add test for nil link and shard

* rename test

* Update CHANGELOG.md

---------

Co-authored-by: Lucas Molas <schomatis@gmail.com>
Co-authored-by: Andrew Gillis <11790789+gammazero@users.noreply.github.com>
Co-authored-by: Andrew Gillis <11790789+gammazero@users.noreply.github.com>
Co-authored-by: Marcin Rataj <lidel@lidel.org>
feat: add routing filtering to delegated routing server IPIP-484
* Do not fetch size for WantHave blocks
* Option to set replaceHasWithBlockMaxSize
* Log if replace logic enabled/disabled
* docs: WithWantHaveReplaceSize

---------

Co-authored-by: Marcin Rataj <lidel@lidel.org>
* feat: add filtering on client
* refactor: abstract add filters to url function
* feat: add client filtering to FindPeers
* test: test filtering in findPeers

---------

Co-authored-by: Daniel N <2color@users.noreply.github.com>
Co-authored-by: gammazero <11790789+gammazero@users.noreply.github.com>
* Update to latest go-libp2p
@gammazero gammazero added release Merging this PR will create a tagged release skip/changelog labels Oct 3, 2024
Copy link
Member

@lidel lidel left a comment

Choose a reason for hiding this comment

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

@gammazero lgtm, (updated changelog :))

Copy link

codecov bot commented Oct 3, 2024

Codecov Report

Attention: Patch coverage is 78.82353% with 72 lines in your changes missing coverage. Please review.

Project coverage is 60.32%. Comparing base (41b8882) to head (eac6c25).
Report is 31 commits behind head on release.

Files with missing lines Patch % Lines
routing/http/filters/filters.go 82.51% 18 Missing and 7 partials ⚠️
routing/http/client/client.go 51.42% 15 Missing and 2 partials ⚠️
...swap/server/internal/decision/blockstoremanager.go 64.51% 8 Missing and 3 partials ⚠️
bitswap/server/server.go 0.00% 7 Missing ⚠️
bitswap/server/internal/decision/engine.go 92.30% 3 Missing and 1 partial ⚠️
routing/http/server/server.go 90.90% 2 Missing and 1 partial ⚠️
routing/http/types/iter/filter.go 86.36% 2 Missing and 1 partial ⚠️
bitswap/options.go 0.00% 2 Missing ⚠️

Impacted file tree graph

@@             Coverage Diff             @@
##           release     #683      +/-   ##
===========================================
+ Coverage    60.26%   60.32%   +0.06%     
===========================================
  Files          241      243       +2     
  Lines        30707    31004     +297     
===========================================
+ Hits         18505    18703     +198     
- Misses       10555    10634      +79     
- Partials      1647     1667      +20     
Files with missing lines Coverage Δ
ipld/unixfs/hamt/hamt.go 76.66% <100.00%> (+0.19%) ⬆️
routing/http/types/record_peer.go 83.63% <100.00%> (+2.78%) ⬆️
bitswap/options.go 36.36% <0.00%> (-1.74%) ⬇️
routing/http/server/server.go 74.08% <90.90%> (+1.61%) ⬆️
routing/http/types/iter/filter.go 86.36% <86.36%> (ø)
bitswap/server/internal/decision/engine.go 91.44% <92.30%> (-0.35%) ⬇️
bitswap/server/server.go 64.77% <0.00%> (+0.13%) ⬆️
...swap/server/internal/decision/blockstoremanager.go 85.33% <64.51%> (-5.43%) ⬇️
routing/http/client/client.go 70.75% <51.42%> (-2.26%) ⬇️
routing/http/filters/filters.go 82.51% <82.51%> (ø)

... and 11 files with indirect coverage changes

(cherry picked from commit a2fc8e7)
@gammazero gammazero marked this pull request as ready for review October 3, 2024 18:57
@gammazero gammazero requested a review from a team as a code owner October 3, 2024 18:57
Copy link

github-actions bot commented Oct 3, 2024

Suggested version: 0.24.0

Comparing to: v0.23.0 (diff)

Changes in configuration file(s):

(empty)

gorelease says:

# diagnostics
go.mod: the following requirements are needed
	go.opentelemetry.io/otel/exporters/jaeger@v1.14.0
Run 'go mod tidy' to add missing requirements.
required module github.com/microcosm-cc/bluemonday@v1.0.1 retracted by module author: Retract older versions as only latest is to be depended upon

# summary
Suggested version: v0.23.1

gocompat says:

HEAD is now at 41b888218 Merge pull request #669 from ipfs/release-v0.23.0
Switched to branch 'release'
Your branch is up to date with 'origin/release'.

Cutting a Release (and modifying non-markdown files)

This PR is modifying both version.json and non-markdown files.
The Release Checker is not able to analyse files that are not checked in to release. This might cause the above analysis to be inaccurate.
Please consider performing all the code changes in a separate PR before cutting the release.

Automatically created GitHub Release

A draft GitHub Release has been created.
It is going to be published when this PR is merged.
You can modify its' body to include any release notes you wish to include with the release.

@gammazero gammazero merged commit 2fdde02 into release Oct 3, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release Merging this PR will create a tagged release skip/changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants