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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: exclude test WebAssembly from linguist #8868

Merged
merged 4 commits into from
Apr 3, 2023
Merged

fix: exclude test WebAssembly from linguist #8868

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a8ba350
impl
pugachAG Apr 3, 2023
0c15b60
ignore all wasm file types
pugachAG Apr 3, 2023
8054f4c
improve comment
pugachAG Apr 3, 2023
240dc96
Merge branch 'master' into fix-displayed-project-lang
pugachAG Apr 3, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .gitattributes
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
**/package-lock.json linguist-generated=true -diff
/runtime/near-vm/tests/wast/spec/* linguist-detectable=false
Copy link
Collaborator

Choose a reason for hiding this comment

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

I’d just ignore anything with **.wast, **.wat and **.wasm extensions. Given the nature of the project(s) these files can end up in other places just as well quite easily, leading to this same issue appearing again.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

that makes sense, done