Skip to content
This repository was archived by the owner on Jun 3, 2024. It is now read-only.

remove publish scripts #622

Merged
merged 6 commits into from
Aug 28, 2019
Merged

remove publish scripts #622

merged 6 commits into from
Aug 28, 2019

Conversation

Marc-Andre-Rivet
Copy link
Contributor

@Marc-Andre-Rivet Marc-Andre-Rivet commented Aug 27, 2019

These scripts are intended to make publishing more predictable / error-proof, but in reality they mostly get in the way and cause unpredictable behavior at publish time.

@Marc-Andre-Rivet Marc-Andre-Rivet marked this pull request as ready for review August 27, 2019 23:28
CONTRIBUTING.md Outdated
@@ -35,12 +35,10 @@ respectively. TODO:
* Add `rc1` to `version.py` (`./dash_core_components/version.py`) e.g. `0.13.0rc1`
* Add `-rc1` to `package.json` e.g. `0.13.0-rc1`
* Update the `unpkg` link in `./dash_core_components/__init__.py`, replacing `__version__` with your release candidate (e.g. `"0.13.0-rc1"`)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we still want this stuff about prereleases? We've kind of moved away from that model.

Actually I'm wondering if any of this "Publishing New Components/Features" section belongs in the component repos anymore. Since all their versions are locked to Dash, seems like we should move these instructions into the CONTRIBUTING.md in dash instead, as a full step-by-step for first publishing the components - including the whole flow about merging to master, committing bundles there, any sanity checks to make sure these bundles are correct, and how to tag the published version so it can be picked up by Dash for R - and then how to update dash itself and include the new versions of the components.


1. Create a pull request and tag the Plotly team (`@plotly/dash`) and tag / request review from [@Marc-Andre-Rivet](https://github.com/Marc-Andre-Rivet) and [@alexcjohnson ](https://github.com/alexcjohnson).
2. After a review has been done and your changes have been approved, they will be merged and included in a future release of Dash.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updating contribution guideline everywhere to... open an issue / open a PR, that's it!

Copy link
Collaborator

@alexcjohnson alexcjohnson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💃

@Marc-Andre-Rivet Marc-Andre-Rivet merged commit 60cf062 into dev Aug 28, 2019
@Marc-Andre-Rivet Marc-Andre-Rivet deleted the update-publish branch August 28, 2019 17:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants