Skip to content

Conversation

erizocosmico
Copy link
Contributor

Closes #337

@ajnavarro ajnavarro requested a review from jfontan June 26, 2018 15:10
@jfontan
Copy link
Contributor

jfontan commented Jun 27, 2018

LGTM but I believe that we can simplify this and do it for all OSs. It can happen that we copy repos from MAC to Linux and have the same problem.

@erizocosmico
Copy link
Contributor Author

Yep, we can probably unify this

@ajnavarro
Copy link
Contributor

should we ignore all hidden files and folders? WDYT @jfontan @erizocosmico

@erizocosmico
Copy link
Contributor Author

erizocosmico commented Jun 27, 2018

Regarding the comment from @ajnavarro in #341: we cannot ignore all hidden files and folders by default. They may have some meaning in the repository. Take into account .travis.yml and the bazillion .something JavaScript projects have.
This is some macOS filesystem file that gets created there. Only problem is we can't distinguish when .DS_Store is or is not committed to the repo (because it might be).

@erizocosmico
Copy link
Contributor Author

Shall we merge this?

@ajnavarro
Copy link
Contributor

@erizocosmico rebase please

Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
@erizocosmico
Copy link
Contributor Author

Done

@ajnavarro
Copy link
Contributor

@erizocosmico yes please.

@erizocosmico erizocosmico merged commit fde310e into src-d:master Jul 3, 2018
@erizocosmico erizocosmico deleted the fix/pattern-matches-macos branch July 3, 2018 08:33
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.

3 participants