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: log during syft step #974

Merged
merged 1 commit into from
May 16, 2024
Merged

fix: log during syft step #974

merged 1 commit into from
May 16, 2024

Conversation

ralphbean
Copy link
Member

I found that the syft step may take a non-negligble amount of time in my builds. Currently, it emits nothing to stdout, so it can feel like your build is stuck doing nothing.

By printing here, the user will get a clue as to what is happening.

Before you complete this pull request ...

Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.

I found that the syft step may take a non-negligble amount of time in my
builds. Currently, it emits nothing to stdout, so it can feel like your
build is stuck doing nothing.

By printing here, the user will get a clue as to what is happening.

Signed-off-by: Ralph Bean <[email protected]>
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@ralphbean ralphbean added this pull request to the merge queue May 2, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 2, 2024
@chmeliik chmeliik added this pull request to the merge queue May 7, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 7, 2024
@ralphbean ralphbean added this pull request to the merge queue May 15, 2024
Merged via the queue into main with commit dab9143 May 16, 2024
6 checks passed
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