search

Neargye / semver

Semantic Versioning for modern C++
MIT License
296 stars 32 forks source link

Limitation to `std::uint8_t` might yield an overflow for some projects #31

Closed daixtrose closed 6 months ago

daixtrose commented 2 years ago

Checking the semver rules I am unsure whether a limitation to a value range from 0..255 (i.e. a limitation to std::uint8_t) is

  1. required and
  2. a wise design decision, especially for the patch number

Could you please explain your design decision?

Neargye commented 2 years ago

This limitation will be removed in the new version(0.4.0), there will be full support for semver.

Danielmelody commented 1 year ago

Better to provide an template parameter for custom data precision

JaiganeshKumaran commented 8 months ago

This limitation will be removed in the new version(0.4.0), there will be full support for semver.

I reälly need this. Can you do this now?

Neargye commented 8 months ago

@JaiganeshKumaran now it's std::uint16_t

JaiganeshKumaran commented 8 months ago

Okay, but the vcpkg has not been updated yet. Also, you have not changed the version number in recent commits.

JaiganeshKumaran commented 6 months ago

@Neargye

daixtrose commented 6 months ago

@JaiganeshKumaran you are aware this is an Open Source project for which you did not pay?