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

Dynamic fee types cleanup #14602

Merged
merged 13 commits into from
Oct 9, 2024
Merged

Dynamic fee types cleanup #14602

merged 13 commits into from
Oct 9, 2024

Conversation

dimriou
Copy link
Collaborator

@dimriou dimriou commented Sep 30, 2024

BCFR-965

Update dynamic fees to align with Geth's spec

@dimriou dimriou requested review from a team as code owners September 30, 2024 12:14
@dimriou dimriou requested review from Atrax1 and removed request for a team September 30, 2024 12:14
amit-momin
amit-momin previously approved these changes Oct 1, 2024
Farber98
Farber98 previously approved these changes Oct 2, 2024
@dimriou dimriou enabled auto-merge October 2, 2024 12:54
@dimriou dimriou changed the title Major dynamic fee types cleanup Dynamic fee types cleanup Oct 2, 2024
huangzhen1997
huangzhen1997 previously approved these changes Oct 2, 2024
patrick-dowell
patrick-dowell previously approved these changes Oct 3, 2024
@dimriou dimriou requested a review from a team as a code owner October 8, 2024 08:03
Copy link
Contributor

github-actions bot commented Oct 8, 2024

WF: CI Core#3936f1e

No errors found in this run. 🎉

dimkouv
dimkouv previously approved these changes Oct 8, 2024
Copy link
Contributor

@dimkouv dimkouv left a comment

Choose a reason for hiding this comment

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

ccip-offchain related renames ✅


// HexToInt64 performs the inverse of Int64ToHex
// Returns 0 on invalid input
func HexToInt64(input interface{}) int64 {
Copy link
Contributor

Choose a reason for hiding this comment

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

Where is this used?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Here and a few other places in BHE. I just moved them, nothing new was added.

Copy link
Contributor

Choose a reason for hiding this comment

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

Do they need to be exported?

amit-momin
amit-momin previously approved these changes Oct 8, 2024
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.

8 participants