-
Notifications
You must be signed in to change notification settings - Fork 6
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #9 from yuvipanda/fix-build
Build js file at package time & cleanu sdist
- Loading branch information
Showing
8 changed files
with
71 additions
and
48,114 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -161,3 +161,6 @@ cython_debug/ | |
|
||
# JS is everywhere | ||
node_modules/ | ||
|
||
# Built files should not be in the package | ||
static/dist/ |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,19 @@ | ||
include LICENSE | ||
include README.md | ||
graft jupyterhub_fancy_profiles/templates | ||
graft jupyterhub_fancy_profiles/static | ||
# setuptools-scm puts all tracked files here already. We need to | ||
# prune files we don't want in our sdist, and include generated files | ||
# Ideally we can simply just *include* files we want, but I have given up | ||
# on that for now. | ||
|
||
# Add generated files | ||
graft jupyterhub_fancy_profiles/static/dist | ||
|
||
# Remove JS files that we won't need when installed | ||
exclude src/* | ||
exclude webpack.config.js | ||
|
||
# Remove docs & dev time needs we won't need when installed in sdist | ||
exclude CONTRIBUTING.md | ||
exclude dev-requirements.txt | ||
exclude jupyterhub_config.py | ||
exclude binderhub_config.py | ||
exclude package.json | ||
exclude screenshot.png |
Oops, something went wrong.