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.

Sign up for GitHub

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

Jump to bottom

pylint in travis wrongly test all files containing 'py' in the name #688

Closed
tbeerman opened this issue Feb 16, 2018 · 0 comments
Closed

pylint in travis wrongly test all files containing 'py' in the name #688

tbeerman opened this issue Feb 16, 2018 · 0 comments
Assignees
@tbeerman
Labels
patch Testing
Milestone
1.15.0

Comments

@tbeerman
Copy link
Contributor

Motivation

The travis test script wrongly generates the list of changed files also including files containing 'py' in the name, e.g., "aliases-py27.conf"
@tbeerman tbeerman self-assigned this Feb 16, 2018
@bari12 bari12 added this to the 1.14.10 milestone Feb 16, 2018
tbeerman added a commit to tbeerman/rucio that referenced this issue Feb 16, 2018
@tbeerman
Testing: fix travis pyling changed files rucio#688
4aec32a
This was referenced Feb 16, 2018
Merged
Merged
bari12 added a commit that referenced this issue Feb 16, 2018
@bari12
Merge pull request #690 from tbeerman/patch-688-fix_travis_pylint_cha…
9c1b30f 
…nged_files

Testing: fix travis pyling changed files #688
bari12 added a commit that referenced this issue Feb 16, 2018
@bari12
Merge pull request #691 from tbeerman/patch-688-fix_travis_pylint_cha…
345951b 
…nged_files

Testing: fix travis pyling changed files #688
@bari12 bari12 closed this as completed Feb 16, 2018
@bari12 bari12 modified the milestones: 1.14.12, 1.15.0 Feb 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tbeerman tbeerman

Labels
patch Testing
Projects
None yet
Milestone
1.15.0
Development

No branches or pull requests
2 participants
@tbeerman @bari12