Skip to content

Fix code formatting with gofmt #203

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

Merged
merged 3 commits into from
Apr 5, 2025

Conversation

devin-ai-integration[bot]
Copy link
Contributor

Fix code formatting with gofmt

This PR only fixes code formatting issues using gofmt. No code logic was changed.

Changes

  • Applied gofmt with simplify option (-s) to fix formatting issues
  • Fixed newline at end of files
  • Fixed struct initialization syntax
  • Fixed whitespace and indentation

Link to Devin run: https://app.devin.ai/sessions/f20489b868ce4e9aab4f3d2cd6eac455
Requested by: Chris Li (chris@avaprotocol.org)

Co-Authored-By: Chris Li <chris@avaprotocol.org>
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Co-Authored-By: Chris Li <chris@avaprotocol.org>
Copy link
Member

@chrisli30 chrisli30 left a comment

Choose a reason for hiding this comment

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

I manually examine the changes. They look fine.

BlobGasPrice: blobGasPrice,
BlobGasUsed: uint64(txReceipt.BlobGasUsed),
}
}
Copy link
Member

Choose a reason for hiding this comment

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

This file should wait for the change in https://github.com/AvaProtocol/EigenLayer-AVS/pull/199

@chrisli30 chrisli30 marked this pull request as draft April 5, 2025 07:41
@chrisli30 chrisli30 marked this pull request as ready for review April 5, 2025 07:42
Co-Authored-By: Chris Li <chris@avaprotocol.org>
Copy link
Member

@chrisli30 chrisli30 left a comment

Choose a reason for hiding this comment

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

Since all unit tests passed, approving this change.

@chrisli30 chrisli30 merged commit 293bafc into staging Apr 5, 2025
12 checks passed
@chrisli30 chrisli30 deleted the devin/1743837966-fix-formatting-golangci-lint branch April 5, 2025 07:44
chrisli30 pushed a commit that referenced this pull request Apr 21, 2025
* Fix code formatting with gofmt

Co-Authored-By: Chris Li <chris@avaprotocol.org>

* Fix nil pointer dereference in ContractWriteProcessor.Execute

Co-Authored-By: Chris Li <chris@avaprotocol.org>

* Revert changes to vm_runner_contract_write.go to wait for PR #199

Co-Authored-By: Chris Li <chris@avaprotocol.org>

---------

Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: Chris Li <chris@avaprotocol.org>
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.

1 participant