FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

exclude tests top-level package, add some JupyterLab trove classifiers by bollwyvl · Pull Request #1513 · bqplot/bqplot · GitHub

/ bqplot Public

exclude tests top-level package, add some JupyterLab trove classifiers - #1513

Merged
martinRenou merged 3 commits into
bqplot:masterfrom
bollwyvl:patch-1
Aug 22, 2022
Merged

exclude tests top-level package, add some JupyterLab trove classifiers#1513
martinRenou merged 3 commits into
bqplot:masterfrom
bollwyvl:patch-1

Conversation

Copy link
Copy Markdown
Contributor

References

Code changes

  • removes the tests top-level module, which conflicts with other distributions
    • as it's "last-in-wins," this is not a reliable place to put them
    • some downstreams like constructor actively choke on them
  • adds some JupyterLab trove classifiers

User-facing changes

  • prospective users would be able to see bqplot in the PyPI browse UI when looking for JupyterLab extensions

Backwards-incompatible changes

  • n/a

Copy link
Copy Markdown
Contributor Author

The lint findings seem irrelevant... seems like a single-purpose PR running some formatters would be more helpful than anything else, with the appropriate git history flags to hide the churn.

martinRenou merged commit ce27e39 into bqplot:master Aug 22, 2022

Copy link
Copy Markdown
Member

Thanks!

Copy link
Copy Markdown
Member

meeseeksdev please backport to 0.12.x

meeseeksmachine pushed a commit to meeseeksmachine/bqplot that referenced this pull request Aug 22, 2022
martinRenou added a commit that referenced this pull request Aug 22, 2022
…3-on-0.12.x

Backport PR #1513 on branch 0.12.x (exclude tests top-level package, add some JupyterLab trove classifiers)
This file contains hidden or 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
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


Back | FazBrowse Home | New Git URL