Skip to content

Delete newline that causes formatting issue on Github.#9054

Merged
mcourteaux merged 5 commits intomainfrom
mcourteaux-pr-template-newline
Mar 17, 2026
Merged

Delete newline that causes formatting issue on Github.#9054
mcourteaux merged 5 commits intomainfrom
mcourteaux-pr-template-newline

Conversation

@mcourteaux
Copy link
Contributor

@mcourteaux mcourteaux commented Mar 17, 2026

Hopefully this was not a precommit auto-fix?

It was. Alex to the rescue.

Hopefully this was not a precommit auto-fix?
@mcourteaux mcourteaux added the skip_buildbots Do not run buildbots on this PR. Must add before opening PR as we scan labels immediately. label Mar 17, 2026
@mcourteaux
Copy link
Contributor Author

@alexreinking Hopefully a config fix in mdformat is possible here.

@mcourteaux
Copy link
Contributor Author

@alexreinking
Copy link
Member

https://github.com/hukkin/mdformat-gfm/?tab=readme-ov-file

We already have that configured...

  - repo: https://github.com/hukkin/mdformat
    rev: 1.0.0
    hooks:
      - id: mdformat
        args: [ --number, --wrap=80 ]
        additional_dependencies:
          - mdformat-gfm
          - mdformat-ruff
          - mdformat-shfmt

@mcourteaux
Copy link
Contributor Author

Don't you have to use the other one as actual repo instead of dependency?

mdformat-gfm doesn't understand GitHub's checklists.
@mcourteaux mcourteaux force-pushed the mcourteaux-pr-template-newline branch from 9f636c0 to 5dde0a7 Compare March 17, 2026 16:30
@alexreinking alexreinking force-pushed the mcourteaux-pr-template-newline branch from 5dde0a7 to 9f636c0 Compare March 17, 2026 16:33
@mcourteaux mcourteaux requested a review from alexreinking March 17, 2026 17:08
@alexreinking
Copy link
Member

Opened hukkin/mdformat-gfm#58

@mcourteaux
Copy link
Contributor Author

Go ahead and approve your own work. I approve.

@mcourteaux mcourteaux merged commit 7e40343 into main Mar 17, 2026
9 of 10 checks passed
@alexreinking alexreinking deleted the mcourteaux-pr-template-newline branch March 17, 2026 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip_buildbots Do not run buildbots on this PR. Must add before opening PR as we scan labels immediately.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants