Skip to content

Commit

Permalink
[HOTFIX] Fix RAT exclusion for known_translations file
Browse files Browse the repository at this point in the history 
Author: Josh Rosen <joshrosen@databricks.com>

Closes #3719 from JoshRosen/rat-fix and squashes the following commits:

1542886 [Josh Rosen] [HOTFIX] Fix RAT exclusion for known_translations file
  • Loading branch information
@JoshRosen
JoshRosen committed Dec 17, 2014
1 parent 4e1112e commit 3d0c37b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .rat-excludes
View file
Original file line number Diff line number Diff line change
Expand Up @@ -64,4 +64,4 @@ dist/*
logs
.*scalastyle-output.xml
.*dependency-reduced-pom.xml
dev/create-release/known_translations
known_translations

4 comments on commit 3d0c37b

@andrewor14
Copy link
Contributor

Choose a reason for hiding this comment

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

I see. Thanks @JoshRosen for the hot fix!

@JoshRosen
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'm surprised that the fully-qualified path didn't work, especially since there are entries like dist/* in this excludes file.

@ddaws
Copy link

@ddaws ddaws commented on 3d0c37b Dec 18, 2014

Choose a reason for hiding this comment

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

@JoshRosen thanks for being awesome and working on such cool and open source system. Here's a coffee @changetip

@changetip
Copy link

Choose a reason for hiding this comment

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

Hi @JoshRosen, I've delivered a Bitcoin tip worth a coffee (4,660 bits/$1.50) from @dreid93 to your ChangeTip wallet.

**Learn about ChangeTip

Please sign in to comment.