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

spelling: "hopeful" #9474

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

spelling: "hopeful" #9474

wants to merge 3 commits into from

Conversation

adamwight
Copy link

Cleans up a small spelling glitch. Separate commits are put in order of increasing risk, as judged by an inexperienced eye.

hopefull -> hopeful, except in the places which could potentially be
used as external interface.
If the trailing underscore means this is an exclusively internal
interface, the change can be safely merged.  But these could
potentially be used directly by downstream code.
This flag seems to be internal, undocumented, and unused.  Any clues?
Copy link
Contributor

github-actions bot commented Feb 23, 2025

CT Test Results

    3 files    141 suites   50m 1s ⏱️
1 602 tests 1 552 ✅ 50 💤 0 ❌
2 311 runs  2 241 ✅ 70 💤 0 ❌

Results for commit 06c252e.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@jhogberg jhogberg added the team:VM Assigned to OTP team VM label Feb 24, 2025
@bjorng bjorng added fix testing currently being tested, tag is used by OTP internal CI labels Feb 24, 2025
@bjorng
Copy link
Contributor

bjorng commented Feb 24, 2025

Thanks for your pull request. All those changes seem to be safe. I've added the branch to our daily builds.

@bjorng bjorng removed the testing currently being tested, tag is used by OTP internal CI label Feb 24, 2025
@adamwight adamwight changed the title [draft] spelling: "hopeful" spelling: "hopeful" Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix team:VM Assigned to OTP team VM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants