Skip to content

Comments

Cleanup#13

Merged
kommendorkapten merged 16 commits intomainfrom
cleanup
Apr 8, 2025
Merged

Cleanup#13
kommendorkapten merged 16 commits intomainfrom
cleanup

Conversation

@kommendorkapten
Copy link
Member

Copilot AI review requested due to automatic review settings April 4, 2025 09:13
@kommendorkapten kommendorkapten requested a review from a team as a code owner April 4, 2025 09:13
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR cleans up existing logging statements by replacing fmt calls with log calls and adds a Docker build & test workflow.

  • Update logging for improved clarity and consistency
  • Add Docker image build/push workflow and a build/test workflow
  • Make minor spelling corrections in log messages and comments

Reviewed Changes

Copilot reviewed 10 out of 12 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
pkg/verifier/verifier.go Replaced fmt with log for error reporting
pkg/verifier/multi.go Updated error logging messages with log functions
pkg/provider/provider.go Consistent log usage in provider validation errors
pkg/authn/provider.go Updated logging related to keychain configuration
cmd/cver/cver.go Logging adjustments in the CLI verification app
cmd/aaop/aaop.go Replaced fmt with log and noted minor spelling issues
.github/workflows/docker.yaml Added workflow to build and push Docker image
.github/workflows/build.yml Added build and test workflow with a typo in a step name
Files not reviewed (2)
  • Dockerfile: Language not supported
  • Makefile: Language not supported

Tip: Leave feedback on Copilot's review comments with the 👎 and 👍 buttons to help improve review quality. Learn more

@kommendorkapten kommendorkapten merged commit 9ad4290 into main Apr 8, 2025
8 checks passed
@kommendorkapten kommendorkapten deleted the cleanup branch April 8, 2025 13:46
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