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

Separate signing of tar and zipballs #33

Merged
merged 3 commits into from
Sep 21, 2020
Merged

Separate signing of tar and zipballs #33

merged 3 commits into from
Sep 21, 2020

Conversation

nichtsfrei
Copy link
Member

What:
Separated signing of tar and zipballs from release to a new sign command.
Fix a bug within cmakeversion that it never sets in_project_dev to false.

Why:

In some cases it makes sense to just create a release without signed zip and tars within it.
When set(PROJECT_DEV 1) was not the last command than it was never correctly set.

How:

Checklist:

create signing of tar and zipballs into an own command 'sign' instead of
enforcing it in release.
before this commit in_project_dev was never set to false and therefore
the actual project_dev change never occurs when there was a command
after set PROJECT_DEV.
@nichtsfrei nichtsfrei marked this pull request as ready for review September 18, 2020 11:07
@codecov
Copy link

codecov bot commented Sep 18, 2020

Codecov Report

Merging #33 into master will increase coverage by 0.76%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #33      +/-   ##
==========================================
+ Coverage   92.34%   93.11%   +0.76%     
==========================================
  Files          26       26              
  Lines        1071     1176     +105     
==========================================
+ Hits          989     1095     +106     
+ Misses         82       81       -1     
Impacted Files Coverage Δ
pontos/release/release.py 95.55% <ø> (+3.08%) ⬆️
pontos/version/cmake_version.py 96.46% <ø> (+0.12%) ⬆️
tests/release/test_release.py 98.77% <ø> (-1.23%) ⬇️
tests/version/test_cmake_version.py 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 766883e...640d401. Read the comment docs.

@y0urself
Copy link
Member

I have seen, that pontos is missing the updated RELEASE.md like in python-gvm here ... can you add that too, and than update it and include the new sign command to it?

@nichtsfrei nichtsfrei merged commit 7ad3f7a into greenbone:master Sep 21, 2020
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