Skip to content

Comments

Fix wrong nil check for SkipBz2#1099

Merged
randombenj merged 3 commits intomasterfrom
fix/1097
Aug 16, 2022
Merged

Fix wrong nil check for SkipBz2#1099
randombenj merged 3 commits intomasterfrom
fix/1097

Conversation

@randombenj
Copy link
Contributor

Requirements

All new code should be covered with tests, documentation should be updated. CI should pass.

Description of the Change

Fixes #1097 wich was introduced with f61514e#diff-be026efeb4b3190a57554a1cff89f6b726ffb0380519517eecd68e8647a89387R214

Checklist

  • unit-test added (if change is algorithm)
  • functional test added/updated (if change is functional)
  • man page updated (if applicable)
  • bash completion updated (if applicable)
  • documentation updated
  • author name in AUTHORS

@codecov
Copy link

codecov bot commented Aug 11, 2022

Codecov Report

Merging #1099 (51ad7be) into master (abf8abb) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master    #1099   +/-   ##
=======================================
  Coverage   66.31%   66.31%           
=======================================
  Files         141      141           
  Lines       15890    15890           
=======================================
  Hits        10538    10538           
  Misses       4603     4603           
  Partials      749      749           
Impacted Files Coverage Δ
api/publish.go 0.00% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@randombenj randombenj merged commit 42cfee2 into master Aug 16, 2022
@neolynx neolynx deleted the fix/1097 branch July 3, 2024 10:17
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.

API publish POST broken with aptly v1.5.0 with 500 internal server errror

1 participant