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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

454 Add .gitattribute file for linguist settings #456

Merged
merged 5 commits into from Jan 10, 2023
Merged

Conversation

martinvonk
Copy link
Collaborator

Short Description

This will hopefully show the correct programming language on GitHub which is essential for searching capabilities on GitHub. I try to get away with moving the .gitattribute file to the .github folder. Hopefully that works, we can only find out after merging 馃檭.

I now do two things:

  1. ignore .pas files (because they are recognized as Pascal)
  2. attribute .ipynb files to the language Python

Checklist before PR can be merged:

ignore .pas file since they are recognized as Pascal language
ignore test folder instead of .pas files
linguist settings based which controls the GitHub language
ignore .pas files
count Jupyter Notebooks as Python files
@raoulcollenteur raoulcollenteur added this to the 0.23.0 milestone Jan 10, 2023
@raoulcollenteur raoulcollenteur marked this pull request as ready for review January 10, 2023 13:05
@raoulcollenteur
Copy link
Member

Looks ready right? Squash and merge if yes!

@martinvonk
Copy link
Collaborator Author

Yes, maybe @dbrakenhoff can have a quick look and judge whether moving the .gitattributes file to the .github folder will work?

@dbrakenhoff
Copy link
Member

I think it has to be the root: https://git-scm.com/book/en/v2/Customizing-Git-Git-Attributes

Copy link
Member

@dbrakenhoff dbrakenhoff left a comment

Choose a reason for hiding this comment

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

Move file to root (see link in comment)

move to root folder
linguist settings based which controls the GitHub language
- ignore .pas files
- count Jupyter Notebooks as Python files
@martinvonk martinvonk merged commit a366d4b into dev Jan 10, 2023
@dbrakenhoff dbrakenhoff deleted the 454-linguist branch January 10, 2023 14:38
@martinvonk martinvonk added the code quality Related to code quality, testing, CI, project setup etc. label Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code quality Related to code quality, testing, CI, project setup etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants