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

fix: Do not panic on invalid air-gapped bundle #586

Merged
merged 1 commit into from
Dec 18, 2023
Merged

Conversation

jimmidyson
Copy link
Contributor

Fixes #514.

Copy link
Contributor

Unit test results

100 tests  ±0   100 ✔️ ±0   0s ⏱️ ±0s
  24 suites ±0       0 💤 ±0 
    1 files   ±0       0 ±0 

Results for commit fa539d1. ± Comparison against base commit 21b5da8.

Copy link
Contributor

e2e test results

  1 files  ±0    2 suites  ±0   1m 31s ⏱️ -40s
33 tests +1  33 ✔️ +1  0 💤 ±0  0 ±0 
35 runs  +1  35 ✔️ +1  0 💤 ±0  0 ±0 

Results for commit fa539d1. ± Comparison against base commit 21b5da8.

@jimmidyson jimmidyson enabled auto-merge (squash) December 18, 2023 13:46
@jimmidyson jimmidyson disabled auto-merge December 18, 2023 15:58
@jimmidyson jimmidyson enabled auto-merge (squash) December 18, 2023 15:58
@jimmidyson jimmidyson merged commit e544b4a into main Dec 18, 2023
13 checks passed
@jimmidyson jimmidyson deleted the jimmi/fix-514 branch December 18, 2023 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

import command panics if invalid bundle is provided
2 participants