Skip to content

Commit

Permalink
Merge dash renderer with DashPy (#1611)
Browse files Browse the repository at this point in the history
* Added dash-renderer artifacts

* Import renderer resources from Dash

* Added gulpfile to update renderer

* Updated gulpfile

* Added newlines

* Another newline fix

* Updating gulpfile for version updates

* Update CHANGELOG

* Flattening package structure and sourcing deps within Dash

* Update package.json scripts

* Updated manifest glob

* Fixed typo

* Added clean job

* Updated build folder and deps

* Added build artifacts to gitignore

* Updating build process and pylintrc

* Running tests with deps

* Update CircleCI config install of renderer

* Updating config.yml

* Update lint-unit tests

* Remove dashr metadata update job from circleci

* Updated manifest

* Remove unnecessary import

* Build cleanup

* Linting exceptions
  • Loading branch information
HammadTheOne authored Jun 1, 2021
1 parent e727dd7 commit 7afb2ed
Show file tree
Hide file tree
Showing 115 changed files with 35,058 additions and 16,217 deletions.
12 changes: 4 additions & 8 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,7 @@ jobs:
command: |
. venv/bin/activate
set -eo pipefail
cd dash/dash-renderer && renderer build && cd ../../
pip install -e . --progress-bar off && pip list | grep dash
npm install --production && npm run initialize
npm run lint
Expand Down Expand Up @@ -121,7 +122,7 @@ jobs:
name: ️️🏗️ build core
command: |
. venv/bin/activate && pip install --no-cache-dir --upgrade -e . --progress-bar off && mkdir packages
cd dash-renderer && renderer build && python setup.py sdist && mv dist/* ../packages/ && cd ..
cd dash/dash-renderer && renderer build && cd ../../ && python setup.py sdist && mv dist/* packages/
git clone --depth 1 https://github.com/plotly/dash-core-components.git
cd dash-core-components && npm ci && npm run build && python setup.py sdist && mv dist/* ../packages/ && cd ..
ls -la packages
Expand Down Expand Up @@ -223,7 +224,7 @@ jobs:
name: ️️🏗️ build core
command: |
pip install --no-cache-dir --upgrade -e .[dev,testing] --progress-bar off
cd dash-renderer && renderer build && python setup.py sdist && mv dist/* ../packages/ && cd ..
cd dash/dash-renderer && renderer build && cd ../../ && python setup.py sdist && mv dist/* packages/
git clone --depth 1 https://github.com/plotly/dash-core-components.git
cd dash-core-components && npm ci && npm run build && python setup.py sdist && cd ..
Expand Down Expand Up @@ -256,12 +257,7 @@ jobs:
cd dash-html-components; npm ci && npm run build; rm -rf !(.|..|DESCRIPTION|LICENSE.txt|LICENSE|NAMESPACE|.Rbuildignore|R|man|inst|vignettes|build)
cd ../dash-core-components; npm ci && npm run build; rm -rf !(.|..|DESCRIPTION|LICENSE.txt|LICENSE|NAMESPACE|.Rbuildignore|R|man|inst|vignettes|build)
cd ../dash-table; npm ci && npm run build; rm -rf !(.|..|DESCRIPTION|LICENSE.txt|LICENSE|NAMESPACE|.Rbuildignore|R|man|inst|vignettes|build); cd ..
- run:
name: 🔧 fix up dash metadata
command: |
sudo Rscript dashR/tests/circleci/fixup_metadata.R
- run:
name: 🎛 set environment variables
command: |
Expand Down
2 changes: 1 addition & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ coverage.xml
# Distribution / packaging
.Python
build/
dash/deps/
develop-eggs/
dist/
downloads/
Expand All @@ -63,7 +64,6 @@ node_modules/
.npm
npm-debug*

dash_renderer/
dash_generator_test_component_standard/
dash_generator_test_component_nested/
dash_test_components/
Expand Down
5 changes: 5 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,11 @@ All notable changes to `dash` will be documented in this file.
This project adheres to [Semantic Versioning](https://semver.org/).

## [UNRELEASED]

## Dash and Dash Renderer
### Changed
- [#1611](https://github.com/plotly/dash/pull/1611) Package dash-renderer artifacts and dependencies with Dash, and source renderer resources from within Dash.

### Fixed
- [#1640](https://github.com/plotly/dash/pull/1640) Fix [#1475](https://github.com/plotly/dash/issues/1475), missing `timing_information` after certain modifications to Flask behavior

Expand Down
2 changes: 2 additions & 0 deletions MANIFEST.in
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,5 @@ include LICENSE
include requires-*.txt
include dash/favicon.ico
include dash/extract-meta.js
include dash/deps/*.js
include dash/deps/*.map
Loading

0 comments on commit 7afb2ed

Please sign in to comment.