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

chore: Fix typos #1964

Merged
merged 4 commits into from
Jan 23, 2024
Merged

chore: Fix typos #1964

merged 4 commits into from
Jan 23, 2024

Conversation

szepeviktor
Copy link
Contributor

@szepeviktor szepeviktor commented Jan 20, 2024

Description

Found many misspellings.

Change checklist

  • Self-review.
  • Documentation updates if relevant.
  • Tests if relevant.

@@ -1018,7 +1018,7 @@ impl DiscoMessageSource {
/// and start a new one when the current one has finished
#[derive(Debug)]
struct EndpointUpdateState {
/// If running, set to the reason fo the currently the update.
/// If running, set to the reason for the currently the update.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This may need DeepL Write or Grammarly.

@flub flub changed the title Fix typos chore: Fix typos Jan 22, 2024
@flub flub enabled auto-merge January 22, 2024 10:18
@flub
Copy link
Contributor

flub commented Jan 22, 2024

Is "entrys" intentional?

"entries" would be the correct spelling

auto-merge was automatically disabled January 22, 2024 11:50

Head branch was pushed to by a user without write access

@szepeviktor
Copy link
Contributor Author

"entries" would be the correct spelling

Done.

@flub flub enabled auto-merge January 22, 2024 11:55
@flub flub added this pull request to the merge queue Jan 22, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 22, 2024
@flub flub enabled auto-merge January 22, 2024 16:12
@flub
Copy link
Contributor

flub commented Jan 22, 2024

#1966 for flaky test

@szepeviktor
Copy link
Contributor Author

.typos.toml

[files]
extend-exclude = [
    ".git/",
    "iroh-net/proptest-regressions/derp/codec.txt",
]
ignore-hidden = false

[default]
extend-ignore-re = [
    "\\[[0-9a-f]{7}\\]",
    "let ba =",
    "if ab && ba",
    '" worl", "d"',
    "Replace `currrent` with `current`",
]

@flub
Copy link
Contributor

flub commented Jan 22, 2024

.typos.toml

[files]
extend-exclude = [
    ".git/",
    "iroh-net/proptest-regressions/derp/codec.txt",
]
ignore-hidden = false

[default]
extend-ignore-re = [
    "\\[[0-9a-f]{7}\\]",
    "let ba =",
    "if ab && ba",
    '" worl", "d"',
    "Replace `currrent` with `current`",
]

@szepeviktor want to contribute that as a CI step? Probably in ci.yml after the cargo_deny step is a good starting point.

In a separate PR of course.

@flub flub added this pull request to the merge queue Jan 23, 2024
@flub flub removed this pull request from the merge queue due to a manual request Jan 23, 2024
@szepeviktor
Copy link
Contributor Author

It seems like a never-ending story ...

@flub
Copy link
Contributor

flub commented Jan 23, 2024

It seems like a never-ending story ...

Yes, apologies for doing all this on your PR. I think I finally found the problem in our CI setup. I'll probably fix this separately in an other PR and then strip all this noise from this PR and rebase it.

@@ -16,7 +16,7 @@ on:
git-ref:
description: 'Which git ref to checkout'
type: string
default: 'main'
default: ${{ github.ref }}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is github.ref_name

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You can browse the github context in this workflow run output https://github.com/szepeviktor/byte-level-care/actions/runs/7585113229/job/20660246214

Copy link
Contributor

Choose a reason for hiding this comment

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

from https://docs.github.com/en/actions/learn-github-actions/contexts i think github.ref points to the same and is more descriptive? Any reason to prefer ref_name?

This is moved to #1971 btw. Let's merge main into this branch here once that's merged.

@flub flub enabled auto-merge January 23, 2024 14:17
@flub flub added this pull request to the merge queue Jan 23, 2024
Merged via the queue into n0-computer:main with commit c2359e8 Jan 23, 2024
18 checks passed
@szepeviktor
Copy link
Contributor Author

Thank you.

@szepeviktor szepeviktor deleted the typos branch January 23, 2024 14:57
fubuloubu pushed a commit to ApeWorX/iroh that referenced this pull request Feb 21, 2024
## Description

Found many misspellings.

## Change checklist

- [x] Self-review.
- [x] Documentation updates if relevant.
- [x] Tests if relevant.

---------

Co-authored-by: Floris Bruynooghe <flub@n0.computer>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants