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

Revert "ci: uci/copy-templates" #450

Closed
wants to merge 1 commit into from
Closed

Conversation

hacdias
Copy link
Member

@hacdias hacdias commented Aug 22, 2023

Reverts #431.

This updates go-libp2p and those updates are not yet ready in Kubo. Since we don't have the reciprocal PR in Kubo, it hinders the development of other PRs since Kubo doesn't build with the latest Boxo, and therefore no commits on top of it.

Unless a PR for Kubo magically shows up within a few hours, I think we should revert this to allow work to continue without major issues.

@hacdias hacdias requested a review from a team as a code owner August 22, 2023 12:11
@codecov
Copy link

codecov bot commented Aug 22, 2023

Codecov Report

Merging #450 (ba2ca23) into main (9e804b4) will decrease coverage by 12.24%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             main     #450       +/-   ##
===========================================
- Coverage   61.37%   49.14%   -12.24%     
===========================================
  Files         205      246       +41     
  Lines       24399    30054     +5655     
===========================================
- Hits        14976    14769      -207     
- Misses       7985    13835     +5850     
- Partials     1438     1450       +12     
Files Changed Coverage Δ
...itswap/testnet/internet_latency_delay_generator.go 85.71% <ø> (ø)
examples/unixfs-file-cid/main.go 41.21% <ø> (ø)

... and 64 files with indirect coverage changes

@galargh
Copy link
Contributor

galargh commented Aug 22, 2023

Are there any boxo consumers, other than Kubo, that might be waiting for go 1.21 compatibility?

@hacdias
Copy link
Member Author

hacdias commented Aug 22, 2023

Are there any boxo consumers, other than Kubo, that might be waiting for go 1.21 compatibility?

@galargh unsure.


I made a PR on Kubo side: ipfs/kubo#10094

I will close this.

@hacdias hacdias closed this Aug 22, 2023
@hacdias hacdias deleted the revert-431-uci/copy-templates branch August 22, 2023 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants