search

enram / vptstools

Python library to transfer and convert vertical profile time series data
https://enram.github.io/vptstools/
MIT License
3 stars 1 forks source link

Update matching version #71

Closed peterdesmet closed 7 months ago

peterdesmet commented 7 months ago

@stijnvanhoey I think there were two issues with the previous regex.

  1. It doesn't match v0.4.0-alpha.1 or even v0.3.0 for that matter.
  2. The second expression is invalid regex (according to regex101).

The current version is more flexible. I did drop the matching for tags that do not start with v

peterdesmet commented 7 months ago

In that case 'v[0-9]*', '[0-9]*' rather than 'v[0-9]*', '[0-9]+.[0-9]+*' should be enough?

stijnvanhoey commented 7 months ago

In that case 'v[0-9]*', '[0-9]*' rather than 'v[0-9]*', '[0-9]+.[0-9]+*' should be enough?

Indeed. Or even only keep the `'v[0-9]*' if that is how you define versions