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

chore: sort reports by results #2036

Merged
merged 1 commit into from
Apr 11, 2024
Merged

chore: sort reports by results #2036

merged 1 commit into from
Apr 11, 2024

Conversation

olblak
Copy link
Member

@olblak olblak commented Apr 11, 2024

UX improvement, this pull request sorts reports results so we first see failed pipelines at the bottom.
This should allows to better identify failed pipelines after an Updatecli execution

Example
⚠ deps(go): bump module go.mozilla.org/sops/v3:
	Source:
		✔ [module] Get latest golang module go.mozilla.org/sops/v3 version
	Target:
		⚠ [module] deps(go): bump module go.mozilla.org/sops/v3 to v3.8.1


⚠ deps(golang): bump Go version:
	Source:
		✔ [go] Get latest Go version
	Target:
		⚠ [go] deps(golang): bump Go version to 1.22.2


✗ deps(go): bump module github.com/nirasan/go-oauth-pkce-code-verifier:
	Source:
		✗ [module] Get latest golang module github.com/nirasan/go-oauth-pkce-code-verifier version
	Target:
		- [module] 
		- [tidy] 


✗ deps(go): bump module golang.org/x/exp:
	Source:
		✗ [module] Get latest golang module golang.org/x/exp version
	Target:
		- [module] 
		- [tidy] 

Test

To test this pull request, you can run the following commands:

cd <to_package_directory>
go test

Additional Information

Tradeoff

Potential improvement

@olblak olblak added the ux label Apr 11, 2024
Signed-off-by: Olblak <me@olblak.com>
@olblak olblak added the enhancement New feature or request label Apr 11, 2024
@olblak olblak merged commit 66c09e2 into updatecli:main Apr 11, 2024
6 checks passed
@olblak olblak deleted the sortReports branch April 11, 2024 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ux
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant