Skip to content
This repository has been archived by the owner on Dec 12, 2024. It is now read-only.

feat: Ignore node_modules by default for processing #57

Merged
merged 1 commit into from
Jul 3, 2024

Conversation

mydea
Copy link
Member

@mydea mydea commented Jul 3, 2024

@andreiborza noticed this, we should not run on node_modules actually... This is usually covered by leveraging gitignore, but this does not seem to always work when working in a monorepo subpackage. I think adding this as default makes sense.

@mydea mydea requested review from Lms24 and andreiborza July 3, 2024 09:43
@mydea mydea self-assigned this Jul 3, 2024
Copy link
Member

@andreiborza andreiborza left a comment

Choose a reason for hiding this comment

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

thanks for adding this!

@mydea mydea merged commit 88b5645 into main Jul 3, 2024
3 checks passed
@mydea mydea deleted the fn/ignore-node-modules branch July 3, 2024 13:29
@mbrevda
Copy link

mbrevda commented Jul 3, 2024

does this address #54?

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.

3 participants