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

Ignore the .git folder for the dev watcher #515

Merged
merged 1 commit into from
Jun 8, 2017

Conversation

Kimtaro
Copy link
Contributor

@Kimtaro Kimtaro commented Jun 8, 2017

No description provided.

Copy link
Contributor

@nickbalestra nickbalestra left a comment

Choose a reason for hiding this comment

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

LGTM!

false
/node_modules|package\.tar\.gz|_package|\.sw[op]|\.git/gi.test(
fileName
) === false
Copy link
Member

Choose a reason for hiding this comment

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

just a small comment: interesting indentation 😅

Copy link
Member

Choose a reason for hiding this comment

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

is it prettier? :D @nickbalestra can we hack around prettier rules to make that...prettier?

Copy link
Member

Choose a reason for hiding this comment

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

@matteofigus this is not prettier this is figus-ier 😄

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I prefer this prettified version over the original since it puts the === on the same line as the false.

@matteofigus matteofigus merged commit c8fb013 into master Jun 8, 2017
@matteofigus matteofigus deleted the ignore_git_directory branch June 8, 2017 15:54
@matteofigus
Copy link
Member

@Kimtaro thanks for your quick PR 👍 I just published oc@0.37.9 that contains this change

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.

4 participants