Fix patterns - too general #921

Closed
wants to merge 2 commits into
from

2 participants

@torbensky

The original ignore was far too general and results in eliminating packages named "db" or "project", for example. These are not that uncommon for package names. Also added a lot of other common, Playframework project related ignore patterns (Scala/Java/sbt/Eclipse).

torbensky added some commits Jan 14, 2014
@torbensky torbensky Fix patterns from
The original ignore was far too general and results in eliminating packages named "db" or "project", for example. These are not that uncommon for package names. Also added a lot of other common, Playframework project related ignore patterns (Scala/Java/sbt/Eclipse).
2192516
@torbensky torbensky TW: Added missing bin/ ignore. d5ed9cd
@arcresu

Thanks, but could you remove the IDE/OS-specific rules? They belong in the various global templates like Global/Linux and Global/Eclipse rather than being duplicated in each template, as per out contributing guidelines.

@arcresu arcresu added a commit that closed this pull request Mar 21, 2014
@torbensky torbensky Increase the specificity of PlayFramework rules
The original ignore was far too general and results in eliminating packages
named "db" or "project", for example. These are not that uncommon for package
names. Fixes #921.
a6cabef
@arcresu arcresu closed this in a6cabef Mar 21, 2014
@arcresu arcresu added a commit that referenced this pull request Mar 21, 2014
@arcresu arcresu Merge branch 'fix-921'
This is #921 without the addition of editor-specific rules.
655e5ba
@drothmaler drothmaler pushed a commit to drothmaler/gitignore that referenced this pull request May 27, 2014
@torbensky torbensky Increase the specificity of PlayFramework rules
The original ignore was far too general and results in eliminating packages
named "db" or "project", for example. These are not that uncommon for package
names. Fixes #921.
a83491e
@drothmaler drothmaler pushed a commit to drothmaler/gitignore that referenced this pull request May 27, 2014
@arcresu arcresu Merge branch 'fix-921'
This is #921 without the addition of editor-specific rules.
b5f440f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment