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

Parse with packaging version #3815

Merged
merged 1 commit into from
Sep 4, 2020

Conversation

bjlittle
Copy link
Member

@bjlittle bjlittle commented Sep 4, 2020

This PR uses packaging.version to safely parse version strings.

See here for a fuller (opinionated) discussion.

@trexfeathers
Copy link
Contributor

@bjlittle thanks, this hadn't occurred to me. Looking forward to Matplotlib version 10!

Copy link
Contributor

@trexfeathers trexfeathers left a comment

Choose a reason for hiding this comment

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

Problem!

@bjlittle bjlittle force-pushed the use-packaging-version-parse branch from b77c18a to b8b18df Compare September 4, 2020 13:37
@trexfeathers
Copy link
Contributor

trexfeathers commented Sep 4, 2020

@bjlittle are you happy with us merging a PR that includes a rebase, despite @abooton 's misgivings about this? I'm still not clear exactly where we stand on this.

@bjlittle
Copy link
Member Author

bjlittle commented Sep 4, 2020

I don't see the problem myself... all I've done is used fixup to join two of my own commits together, and everything is atop the last commit on master.

I personally have no problem with rebasing (for simple workflows).

@trexfeathers
Copy link
Contributor

I don't see the problem myself... all I've done is used fixup to join two of my own commits together, and everything is atop the last commit on master.

I personally have no problem with rebasing.

Understood

@trexfeathers trexfeathers merged commit 1f6627a into SciTools:master Sep 4, 2020
@bjlittle
Copy link
Member Author

bjlittle commented Sep 4, 2020

@trexfeathers Awesome, thanks 👍

BTW For an interesting read, see here

@bjlittle bjlittle deleted the use-packaging-version-parse branch September 4, 2020 14:09
tkknight added a commit to tkknight/iris that referenced this pull request Sep 7, 2020
…haul

* upstream/master:
  move whatsnew contributions (SciTools#3816)
  Parse with packaging version (SciTools#3815)
  Add "What's New" entries for unpinning Cartopy, Matplotlib, Proj (SciTools#3811)
  tidy issue templates (SciTools#3814)
  Legacy doc notice (SciTools#3813)
  Update issue templates
  Update issue templates
  Update issue templates
  Unpin Matplotlib and proj (SciTools#3762)
tkknight added a commit to tkknight/iris that referenced this pull request Sep 8, 2020
* master:
  Add template collapsible markdown section (SciTools#3823)
  Requirements re-haul (SciTools#3812)
  Automate pull request labels (SciTools#3819)
  move whatsnew contributions (SciTools#3816)
  Parse with packaging version (SciTools#3815)
  Add "What's New" entries for unpinning Cartopy, Matplotlib, Proj (SciTools#3811)
  tidy issue templates (SciTools#3814)
  Legacy doc notice (SciTools#3813)
tkknight added a commit to tkknight/iris that referenced this pull request Sep 11, 2020
* master: (36 commits)
  Rework whatsnew into new scheme. (SciTools#3834)
  Lazy regridding with Linear, Nearest, and AreaWeighted (SciTools#3701)
  Iris readme minimal (SciTools#3833)
  Fix sphinx logger (SciTools#3832)
  whatsnew entry for dependencies (SciTools#3831)
  Added link to scitools.org.uk on the main index. (SciTools#3830)
  Updated Iris install instructions (SciTools#3817)
  Delete labeler.yml
  Delete label.yml
  Create label.yml
  Delete labeler.yml
  Update labeler.yml
  Update labeler.yml
  Add template collapsible markdown section (SciTools#3823)
  Requirements re-haul (SciTools#3812)
  Automate pull request labels (SciTools#3819)
  move whatsnew contributions (SciTools#3816)
  Parse with packaging version (SciTools#3815)
  Add "What's New" entries for unpinning Cartopy, Matplotlib, Proj (SciTools#3811)
  tidy issue templates (SciTools#3814)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants