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

Fix spelling errors in l1_gas_price.rs, trace.rs, l1_db.rs, and client.rs #481

Closed
wants to merge 4 commits into from

Conversation

Pronoss
Copy link

@Pronoss Pronoss commented Jan 29, 2025

Pull Request Type

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Build-related changes
  • Documentation content changes
  • Testing
  • Other (please describe):

What Is the Current Behavior?

This PR corrects multiple spelling errors in comments and function names across different files, improving code clarity and consistency.

What Is the New Behavior?

  • l1_db.rs:

    • Fixed occuredoccurred in a comment about L1 → L2 message events.
  • client.rs:

    • Fixed occurenceoccurrence in a comment about checking LogStateUpdate.
  • l1_gas_price.rs:

    • Fixed timestemptimestamp in variable naming for gas price updates.
  • trace.rs:

    • Fixed agregate_execution_ressourcesaggregate_execution_ressources in function definitions and calls.

Does This Introduce a Breaking Change?

  • Yes
  • No

Other Information

These changes only affect comments and function names, ensuring better readability without modifying core functionality.

Checklist

  • Performed a self-review of the changes.
  • Opened or linked an existing issue if applicable.

@Trantorian1
Copy link
Collaborator

Prs which only provide code spelling changes will be rejected. You are welcome to contribute by opening prs with logic-related changes or filing an issue.

@madara-alliance madara-alliance locked as spam and limited conversation to collaborators Jan 30, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants