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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Exclude generated yarn.lock files #5681

Closed
wants to merge 1 commit into from
Closed

Conversation

colinwm
Copy link
Contributor

@colinwm colinwm commented Dec 10, 2021

No description provided.

@colinwm colinwm requested a review from a team as a code owner December 10, 2021 16:49
@lildude
Copy link
Member

lildude commented Dec 10, 2021

We deliberately don't mark these as generated and our test has caught your attempt to change that 😁. See #4348 for more details as to why.

Closing.

@lildude lildude closed this Dec 10, 2021
@colinwm
Copy link
Contributor Author

colinwm commented Dec 10, 2021

@lildude Oops! Missed that discussion, sorry about that. But it clearly is generated... I guess the purpose of linguist is more to identify what should be shown in diffs than to categorize code?

@lildude lildude deleted the colinwm-yarn-lock branch July 19, 2023 10:58
@github-linguist github-linguist locked as resolved and limited conversation to collaborators Jun 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants