Skip to content

Conversation

@jeroenvervaeke
Copy link
Member

Proposed changes

  • Upgrade golinterci
  • Fix warnings

Jira ticket: CLOUDP-300869

@jeroenvervaeke jeroenvervaeke requested a review from a team as a code owner February 14, 2025 17:49
assert.True(t, *setting.Enabled)
assert.True(t, *setting.AuditAuthorizationSuccess)
assert.Equal(t, "{\"atype\": \"authenticate\"}", *setting.AuditFilter)
assert.JSONEq(t, "{\"atype\": \"authenticate\"}", *setting.AuditFilter)
Copy link
Contributor

Choose a reason for hiding this comment

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

TIL

@apix-bot
Copy link
Contributor

apix-bot bot commented Feb 18, 2025

Coverage Report 📈

Branch Commit Coverage
master ab76bb3 20.4%
CLOUDP-300869 3884c29 20.4%
Difference 0%

@jeroenvervaeke jeroenvervaeke enabled auto-merge (squash) February 18, 2025 11:58
@gssbzn
Copy link
Contributor

gssbzn commented Feb 18, 2025

evergreen keep-definitions

Copy link
Contributor

@gssbzn gssbzn left a comment

Choose a reason for hiding this comment

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

since you are updating go we depend on it being available on evergreen, adding the packaging tests which uses other vms like windows to make sure go1.24 is available, good to merge if packaging passes

@jeroenvervaeke jeroenvervaeke merged commit df33d20 into master Feb 18, 2025
22 checks passed
@jeroenvervaeke jeroenvervaeke deleted the CLOUDP-300869 branch February 18, 2025 12:01
blva added a commit that referenced this pull request Feb 18, 2025
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.

3 participants