Skip to content

Convert conda packages back noarch format #1453

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Mar 6, 2019
Merged

Conversation

jonmmease
Copy link
Contributor

@jonmmease jonmmease commented Mar 6, 2019

Closes #1414

Also run plotlyjs_dev_build on all branches so that it's possible to QA plotly.py branches built against plotly.js upstream

@jonmmease jonmmease merged commit 6ae0837 into master Mar 6, 2019
@@ -278,14 +257,46 @@ of the dependencies. Use the release candidate versions, this way we can point
people to the README of the `release_X.Y.Z` as the instructions for trying out
the release candidate.

Note that the conda installation instructions must include
"-c plotly/lable/test" rather than "-c plotly" in order to install the
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nothing is standing out to me. Could you explain what you are noticing?

Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry, I was in a rush - it says "lable" instead of, I guess, "label"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Wow, thanks!

@nicolaskruchten nicolaskruchten deleted the maint_conda_noarch branch June 19, 2020 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants