facebook / akd

An implementation of an auditable key directory
Apache License 2.0
246 stars 36 forks source link

Support pre-release version numbers in deploy script #431

Closed afterdusk closed 8 months ago

afterdusk commented 8 months ago

Updates our deploy script to handle pre-release version strings. Without it, our deploy script currently hangs with the following log:

...
Available akd_core versions: "0.8.0" "0.8.2" "0.8.3" "0.8.4" "0.8.5" "0.8.6" "0.8.7" "0.8.8" "0.8.9" "0.9.0" "0.10.0" "0.11.0" "0.12.0-pre.1" 
Expected version of akd_core (0.12.0) is not yet published. Retrying after a wait
...

Also bumps our version to 0.12.0-pre.2 to re-attempt deploy.

Regex Test


% echo "0.11" | grep -o '[0-9]\.[0-9\.]\+\(-pre\.[0-9]\+\)\?'  
0.11

% echo "0.11.1" | grep -o '[0-9]\.[0-9\.]\+\(-pre\.[0-9]\+\)\?' 
0.11.1

% echo "0.11.11" | grep -o '[0-9]\.[0-9\.]\+\(-pre\.[0-9]\+\)\?' 
0.11.11

% echo "0.11-pre.1" | grep -o '[0-9]\.[0-9\.]\+\(-pre\.[0-9]\+\)\?' 
0.11-pre.1

% echo "0.11.0-pre.12" | grep -o '[0-9]\.[0-9\.]\+\(-pre\.[0-9]\+\)\?' 
0.11.0-pre.12
codecov-commenter commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.57%. Comparing base (493c9bc) to head (274bb88).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #431 +/- ## ========================================== - Coverage 88.58% 88.57% -0.02% ========================================== Files 39 39 Lines 9077 9077 ========================================== - Hits 8041 8040 -1 - Misses 1036 1037 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.