Skip to content
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

DOC Update docs for iodide code removal #981

Merged
merged 2 commits into from Dec 31, 2020
Merged

Conversation

dalcde
Copy link
Contributor

@dalcde dalcde commented Dec 31, 2020

Relevant iodide PR to minimize breakage: iodide-project/iodide#3015

Relevant iodide PR to minimize breakage:
iodide-project/iodide#3015
Copy link
Member

@rth rth left a comment

Choose a reason for hiding this comment

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

There is a few more occurrences;

docs/index.rst
26:Pyodide may be used in several ways, including in an Iodide notebook, directly
35:   using_pyodide_from_iodide.md

docs/type_conversions.md
8:  (evaluating a Python cell in Iodide)

docs/loading_packages.md
60:For use in Iodide:

Ideally we should find a way to deploy docs on readthedocs for v0.16.1 that would not be overwritten by latest development changes, so that iodide users would still have docs to refer to. We would need to get manainer's right on the RDT project for this though.

Edit: though linking to the docs on github as you did in iodide is also probably fine

*Unreleased*

- Removed iodide-specific code in `pyodide.js`. This breaks compatibility with
iodide and other users pretending to be iodide.
Copy link
Member

Choose a reason for hiding this comment

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

I think

Suggested change
iodide and other users pretending to be iodide.
iodide.

would be fine. Could we maybe add a badge "Breaking change" in the beginning as in https://scikit-learn.org/stable/whats_new/v0.24.html ?

@dalcde
Copy link
Contributor Author

dalcde commented Dec 31, 2020 via email

@rth rth changed the title Update docs for iodide code removal DOC Update docs for iodide code removal Dec 31, 2020
@rth rth merged commit c4548db into pyodide:master Dec 31, 2020
### Breaking changes

- Removed iodide-specific code in `pyodide.js`. This breaks compatibility with
iodide.
Copy link
Member

Choose a reason for hiding this comment

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

For future PRs, it's also helpful to include a link to the PR in the changelog entries. (I'll fix this one in #987)

joemarshall pushed a commit to joemarshall/pyodide that referenced this pull request Jan 3, 2021
@dalcde dalcde deleted the iodide-docs branch January 10, 2021 02:56
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.

None yet

2 participants