Skip to content

Commit

Permalink
Add explicit deps from manage.py to settings.py in each app. (pantsbu…
Browse files Browse the repository at this point in the history
…ild#31)

With this change, the example commands in the README work properly
(e.g., pants --concurrent run helloworld/service/frontend/manage.py -- runserver)

Also explicitly note the buildroot, now that `./pants` is no more.
  • Loading branch information
benjyw authored Mar 23, 2023
1 parent ba3fb9a commit 67681a5
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 0 deletions.
Empty file added BUILDROOT
Empty file.
3 changes: 3 additions & 0 deletions helloworld/service/frontend/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,9 @@ python_sources(
dependencies=[
"helloworld/ui",
],
overrides={
"manage.py": {"dependencies": ["helloworld/service/frontend/settings.py:lib"]}
},
)

pex_binary(
Expand Down
3 changes: 3 additions & 0 deletions helloworld/service/user/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,9 @@ python_sources(
dependencies=[
"helloworld/person",
],
overrides={
"manage.py": {"dependencies": ["helloworld/service/user/settings.py:lib"]}
},
)

pex_binary(
Expand Down
3 changes: 3 additions & 0 deletions helloworld/service/welcome/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,9 @@ python_sources(
"helloworld/greet",
"helloworld/translate",
],
overrides={
"manage.py": {"dependencies": ["helloworld/service/welcome/settings.py:lib"]}
},
)

pex_binary(
Expand Down

0 comments on commit 67681a5

Please sign in to comment.