Skip to content

update .gitattributes#9299

Open
DoStupidThingsFasterWithMoreEnergy wants to merge 2 commits into
ovh:developfrom
DoStupidThingsFasterWithMoreEnergy:gitattribute
Open

update .gitattributes#9299
DoStupidThingsFasterWithMoreEnergy wants to merge 2 commits into
ovh:developfrom
DoStupidThingsFasterWithMoreEnergy:gitattribute

Conversation

@DoStupidThingsFasterWithMoreEnergy
Copy link
Copy Markdown
Contributor

What type of Pull Request is this?

  • Optimization

Description

I noticed that @tcpdumpfbacke (I really like that username) has updated the .gitattribute file, and if you’re happy with that, I’d like to contribute to it

Given my choice of linguist-documentation=false for *.md files, please refer to the documentation available here: https://github.com/github-linguist/linguist/blob/main/docs/overrides.md

Mandatory information

The translations in this Pull Request have been done using:

  • This Pull Request didn't require any translation.

  • This Pull Request can be merged as soon as possible.

@Y0Coss
Copy link
Copy Markdown
Collaborator

Y0Coss commented Apr 30, 2026

Hi, thanks for the PR and for linking the Linguist docs!

You're right that *.md linguist-documentation=false makes sense here — Markdown really is the product in this repo, so it should count in the language stats. Nice catch.

Would you mind trimming the rest of the changes though? A few things don't quite fit this repo:

  • The rules for *.py, *.css, *.sh, *.json, etc. — we don't really have those file types here, so they don't do much.
  • The export-ignore rules — we don't ship release tarballs, so they have no effect.
  • Our current .gitattributes is intentionally minimal (just LF on *.md).

Could you keep just the LF normalization + your *.md linguist-documentation=false line, and drop the rest? Also a trailing newline at the end would be great.

Thanks a lot! 🙏

@DoStupidThingsFasterWithMoreEnergy
Copy link
Copy Markdown
Contributor Author

Hi, @Y0Coss,
According to your comment, I just updated the .gitattribute.

I intentionally left some extensions in there because they exist in the repository. Feel free to remove or comment them out.

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