Skip to content

ENH: Add .gitattributes file for linter action - #171

Merged
dzenanz merged 2 commits into
mainfrom
linterGitAttributes
Sep 30, 2026
Merged

dzenanz merged 2 commits into
mainfrom
linterGitAttributes

Conversation

@dzenanz

@dzenanz dzenanz commented Sep 29, 2026

Copy link
Copy Markdown
Member

@dzenanz
dzenanz force-pushed the linterGitAttributes branch 2 times, most recently from ce1adc9 to 1d89e26 Compare September 29, 2026 18:13
See InsightSoftwareConsortium/ITKClangFormatLinterAction#24
File taken from InsightSoftwareConsortium/ITKThinShellDemons#27

Note: the module template no longer passes it, so remove clang-format-linter.yml from here.
@thewtex

thewtex commented Sep 30, 2026

Copy link
Copy Markdown
Member

Why is the Python build / test / package removed?

@dzenanz

dzenanz commented Sep 30, 2026 •

Copy link
Copy Markdown
Member Author

It fails for the cookie cutter template part. Can we maintain it?

@thewtex

thewtex commented Sep 30, 2026

Copy link
Copy Markdown
Member

I think it should be maintained. It is some work, but it will avoid more issues and confusion when people start from a broken template.

@dzenanz

dzenanz commented Sep 30, 2026

Copy link
Copy Markdown
Member Author

I can skip that commit, and merge the rest. Does the rest look OK?

@thewtex

thewtex commented Sep 30, 2026

Copy link
Copy Markdown
Member

Yes, thanks!

@dzenanz
dzenanz force-pushed the linterGitAttributes branch from 54a3760 to ca1c21c Compare September 30, 2026 13:53

@thewtex thewtex left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, @dzenanz !

@dzenanz
dzenanz merged commit 50aef61 into main Sep 30, 2026
4 of 17 checks passed
@dzenanz
dzenanz deleted the linterGitAttributes branch September 30, 2026 15:21
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