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 typo in distinguishingProperty variable name #15763

Merged
merged 2 commits into from
Jan 24, 2025

Conversation

kilavvy
Copy link
Contributor

@kilavvy kilavvy commented Jan 23, 2025

This PR fixes a typo in the OverrideChecker.cpp file where distinguishigProperty was incorrectly spelled (missing n). The variable name has been corrected to distinguishingProperty to maintain consistent naming throughout the codebase.

Changes:

  • Corrected spelling of variable name in error message construction
  • Updated related string concatenation to use proper variable name

Copy link

Thank you for your contribution to the Solidity compiler! A team member will follow up shortly.

If you haven't read our contributing guidelines and our review checklist before, please do it now, this makes the reviewing process and accepting your contribution smoother.

If you have any questions or need our help, feel free to post them in the PR or talk to us directly on the #solidity-dev channel on Matrix.

@cameel cameel enabled auto-merge (squash) January 23, 2025 19:17
@cameel cameel disabled auto-merge January 23, 2025 21:03
@cameel cameel enabled auto-merge (squash) January 24, 2025 02:20
@cameel cameel merged commit 11b770f into ethereum:develop Jan 24, 2025
71 of 74 checks passed
aarlt pushed a commit that referenced this pull request Jan 28, 2025
* Update FilesystemUtils.cpp

* Update OverrideChecker.cpp
aarlt pushed a commit that referenced this pull request Jan 28, 2025
* Update FilesystemUtils.cpp

* Update OverrideChecker.cpp
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