Skip to content

Commit

Permalink
GH-35 cover repo-name-neq-plugin-id.git
Browse files Browse the repository at this point in the history
.ratignore cover artifacts in spec/support/repo-name-neq-plugin-id.git
  • Loading branch information
Christopher J. Brody committed Aug 6, 2018
1 parent 5bc3e1b commit ed14f04
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions .ratignore
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
jasmine.json
testpkg.json
appveyor.yml
HEAD
config
master

5 comments on commit ed14f04

@raphinesse
Copy link
Contributor

Choose a reason for hiding this comment

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

Does rat support ignoring directories? If so, I would prefer ignoring the repo-name-neq-plugin-id.git folder

@brodycj
Copy link

@brodycj brodycj commented on ed14f04 Aug 6, 2018

Choose a reason for hiding this comment

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

Let me try it

@brodycj
Copy link

@brodycj brodycj commented on ed14f04 Aug 6, 2018

Choose a reason for hiding this comment

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

Just adding *.git seems to do the trick, yes. (Initial attempt to use absolute paths, absolute file names, did not work.) Will update PR #36.

@raphinesse
Copy link
Contributor

Choose a reason for hiding this comment

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

I approve this PR with that change

@brodycj
Copy link

@brodycj brodycj commented on ed14f04 Aug 6, 2018

Choose a reason for hiding this comment

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

I approve this PR with that change

Do you mind updating the PR itself? This is just a loose commit that is now obsolete. added reference to ed14f04#commitcomment-29979429

Please sign in to comment.