Skip to content
Projects
Groups
Snippets
Help
This project
Loading...
Sign in / Register
Toggle navigation
C
configuration
Overview
Overview
Details
Activity
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
OpenEdx
configuration
Commits
f8cc3bd3
Commit
f8cc3bd3
authored
Aug 01, 2017
by
syed-awais-ali
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
removed extra print statement
parent
9947bb47
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
with
4 additions
and
3 deletions
+4
-3
util/parsefiles.py
+4
-3
No files found.
util/parsefiles.py
View file @
f8cc3bd3
...
...
@@ -339,7 +339,6 @@ def _get_modified_dockerfiles(files, git_dir):
:return:
"""
items
=
set
()
print
files
candidate_files
=
{
f
for
f
in
DOCKER_PATH_ROOT
.
glob
(
"*/Dockerfile"
)}
for
f
in
files
:
file_path
=
pathlib2
.
Path
(
git_dir
,
f
)
...
...
@@ -359,7 +358,6 @@ def _get_play_name_from_dockerfile(path):
"""
# get individual parts of a file path
dirs
=
path
.
parts
# name of play is the preceeding part of the file path before "Dockerfile"
return
dirs
[
dirs
.
index
(
"Dockerfile"
)
-
1
]
...
...
@@ -422,7 +420,10 @@ if __name__ == '__main__':
docker_plays
=
filter_docker_plays
(
docker_plays
,
TRAVIS_BUILD_DIR
)
# Add playbooks to the list whose docker file has been modified
modified_docer_files
=
_get_modified_dockerfiles
(
change_set
,
TRAVIS_BUILD_DIR
)
modified_docker_files
=
_get_modified_dockerfiles
(
change_set
,
TRAVIS_BUILD_DIR
)
# prints Docker plays
print
" "
.
join
(
str
(
play
)
for
play
in
docker_plays
)
print
" "
.
join
(
str
(
dock
)
for
dock
in
modified_docker_files
)
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment