Skip to content

Fix lint error on main branch#6403

Merged
aduth merged 1 commit intomainfrom
aduth-mfa-context-lint
May 24, 2022
Merged

Fix lint error on main branch#6403
aduth merged 1 commit intomainfrom
aduth-mfa-context-lint

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented May 24, 2022

Why: So that the build passes.

Lint error is legit, but not caught previously due to unfortunate merge timing between #6395 and #6384, and a failure on my part to rebase #6395 with latest main changes prior to merge.

**Why**: So that the build passes

changelog: Internal, Code Quality, Apply and enforce consistent code indentation
@aduth aduth requested a review from SammySteiner May 24, 2022 14:04
Copy link
Contributor

@SammySteiner SammySteiner left a comment

Choose a reason for hiding this comment

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

good catch!

Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM

@aduth aduth merged commit 5e0e9b2 into main May 24, 2022
@aduth aduth deleted the aduth-mfa-context-lint branch May 24, 2022 14:37
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.

3 participants