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.

Sign up for GitHub

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

Jump to bottom

[MM-58039] Enable errcheck linter for enterprise/saml package #26909

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

[MM-58039] Enable errcheck linter for enterprise/saml package #26909

wants to merge 3 commits into from

Conversation

hanzei
Copy link
Contributor

@hanzei hanzei commented Apr 29, 2024 •
edited

@mm-cloud-bot mm-cloud-bot added the release-note-none Denotes a PR that doesn't merit a release note. label Apr 29, 2024
@hanzei hanzei changed the title [MM-58039] Enable errcheck linter for saml package [MM-58039] Enable errcheck linter for enterprise/saml package Apr 29, 2024
@hanzei hanzei marked this pull request as ready for review April 30, 2024 08:13
@hanzei hanzei added 2: Dev Review Requires review by a developer Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation Tests/Not Needed New release tests not required labels Apr 30, 2024
@hanzei hanzei added 4: Reviews Complete All reviewers have approved the pull request Do Not Merge Should not be merged until this label is removed and removed 2: Dev Review Requires review by a developer labels May 1, 2024
@hanzei hanzei added this to the v9.9.0 milestone May 3, 2024
@hanzei hanzei added Do Not Merge/Awaiting PR Awaiting another pull request before merging (e.g. server changes) and removed Do Not Merge Should not be merged until this label is removed labels May 6, 2024
@hanzei hanzei removed this from the v9.9.0 milestone May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sbishel sbishel sbishel approved these changes

@BenCookie95 BenCookie95 BenCookie95 approved these changes

Assignees
No one assigned
Labels
4: Reviews Complete All reviewers have approved the pull request Changelog/Not Needed Does not require a changelog entry Do Not Merge/Awaiting PR Awaiting another pull request before merging (e.g. server changes) Docs/Not Needed Does not require documentation release-note-none Denotes a PR that doesn't merit a release note. Tests/Not Needed New release tests not required
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@hanzei @sbishel @BenCookie95 @mm-cloud-bot