Skip to content

Conversation

@mkatychev
Copy link
Member

related PR: tweag/topiary#832

  • Added typos config to Cargo.toml

    nickel/Cargo.toml

    Lines 147 to 157 in d77b6c3

    # https://github.com/crate-ci/typos/blob/master/docs/reference.md
    [workspace.metadata.typos.default]
    extend-ignore-words-re = [
    '^[a-zA-Z]{1,3}$' # ignore words up to length 3
    ]
    [workspace.metadata.typos.default.extend-words]
    # closure shorthand
    clos = "clos"
    # field shorthand
    fiel = "fiel"
  • Ran typos -w on the project
  • Manually fixed ambiguous resolution typos: d77b6c3

@mkatychev mkatychev marked this pull request as draft October 21, 2025 21:44
@mkatychev
Copy link
Member Author

Not sure where this line/column diff is coming from 🤔 https://github.com/tweag/nickel/actions/runs/18698880001/job/53323091230?pr=2390#step:5:1848

},
{
"id": "605af43a6cace6298b5c47f1",
"id": "605af43a6cache6298b5c47f1",
Copy link
Member Author

Choose a reason for hiding this comment

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

whoops, lmao, will fix

{
"id": 3,
"name": "Selena Shepard"
"name": "Selena Shepherd"
Copy link
Member Author

Choose a reason for hiding this comment

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

revert

@jneem
Copy link
Member

jneem commented Oct 22, 2025

It may be best to redo this after #2302 is finished -- otherwise there will be a lot of conflicts

@mkatychev
Copy link
Member Author

Sounds good, all the manual steps have been documented so I can just rerun all of it.

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.

2 participants