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

Run cargo autoinherit #9041

Merged
merged 1 commit into from
Feb 4, 2025
Merged

Run cargo autoinherit #9041

merged 1 commit into from
Feb 4, 2025

Conversation

oxarbitrage
Copy link
Contributor

Motivation

The CI seems to be stuck in #9030. This is just a temporal replacement that test the changes against the full suite.

It was done by running cargo autoinherit locally and pushing the results. It is the same code changes as #9030

Solution

Tests

Follow-up Work

PR Author's Checklist

  • The PR name will make sense to users.
  • The PR provides a CHANGELOG summary.
  • The solution is tested.
  • The documentation is up to date.
  • The PR has a priority label.

PR Reviewer's Checklist

  • The PR Author's checklist is complete.
  • The PR resolves the issue.

@github-actions github-actions bot added the C-trivial Category: A trivial change that is not worth mentioning in the CHANGELOG label Nov 18, 2024
@oxarbitrage oxarbitrage mentioned this pull request Nov 18, 2024
7 tasks
@oxarbitrage
Copy link
Contributor Author

The 2 failing tests are expected and unrelated. We can merge either #9030 or this one and close the other one.

@oxarbitrage oxarbitrage added A-compatibility Area: Compatibility with other nodes or wallets, or standard rules P-Medium ⚡ labels Nov 19, 2024
@oxarbitrage oxarbitrage marked this pull request as ready for review November 19, 2024 12:11
@oxarbitrage oxarbitrage requested review from a team as code owners November 19, 2024 12:11
@oxarbitrage oxarbitrage requested review from arya2 and upbqdn and removed request for a team November 19, 2024 12:11
@oxarbitrage oxarbitrage added A-dependencies Area: Dependency file updates and removed A-compatibility Area: Compatibility with other nodes or wallets, or standard rules labels Nov 19, 2024
@oxarbitrage
Copy link
Contributor Author

FYI: We are going to wait for Mainnet activation which should be next Saturday 21 of November. So this should be merged probably next week.

@arya2 arya2 added do-not-merge Tells Mergify not to merge this PR and removed do-not-merge Tells Mergify not to merge this PR labels Dec 5, 2024
@oxarbitrage oxarbitrage force-pushed the autoinherit branch 2 times, most recently from 17be5a1 to b97e5c6 Compare December 16, 2024 16:03
@oxarbitrage
Copy link
Contributor Author

This might be a good time to merge this so i updated the PR by executing cargo autoinherit in the last zebra main and force pushed.

@upbqdn upbqdn removed the C-trivial Category: A trivial change that is not worth mentioning in the CHANGELOG label Feb 3, 2025
mergify bot added a commit that referenced this pull request Feb 4, 2025
mergify bot added a commit that referenced this pull request Feb 4, 2025
@mergify mergify bot merged commit d26b0c1 into main Feb 4, 2025
145 checks passed
@mergify mergify bot deleted the autoinherit branch February 4, 2025 03:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-dependencies Area: Dependency file updates P-Critical 🚑
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants