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

fix: fix lint, Break up long regex over multiple lines #116

Merged
merged 1 commit into from
May 28, 2024

Conversation

amarsjac
Copy link
Contributor

What was the problem/requirement? (What/Why)

A long regex was failing linting

What was the solution? (How)

break the regex over multiple lines

What is the impact of this change?

linting should be fixed unblocking all other PR

How was this change tested?

  • hatch -v run lint
  • hatch run test

Was this change documented?

no

Is this a breaking change?

no

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@amarsjac amarsjac requested a review from a team as a code owner May 27, 2024 20:58
Signed-off-by: Marshall Jackson <amarsjac@amazon.com>
@amarsjac amarsjac force-pushed the amarsjac/fix-lint branch from dcc8898 to 3ac65c2 Compare May 27, 2024 21:02
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@amarsjac amarsjac merged commit 3801c80 into OpenJobDescription:mainline May 28, 2024
13 checks passed
@amarsjac amarsjac deleted the amarsjac/fix-lint branch May 28, 2024 18:13
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