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

fix major issues from sonarqube #2336

Open
wants to merge 1 commit into
base: integration
Choose a base branch
from
Open

fix major issues from sonarqube #2336

wants to merge 1 commit into from

Conversation

cogross
Copy link
Collaborator

@cogross cogross commented Apr 3, 2024

@cogross cogross force-pushed the 2321-fix-major-issues-sq branch 3 times, most recently from dd6a584 to 119a61e Compare April 5, 2024 15:57
sparkyy34
sparkyy34 previously approved these changes Apr 5, 2024
View reviewed changes
@cogross cogross changed the title WIP - fix major issues from sonarqube fix major issues from sonarqube Apr 17, 2024
@cogross cogross force-pushed the 2321-fix-major-issues-sq branch 5 times, most recently from 282a3e6 to b1cfb9c Compare April 23, 2024 10:51
@cogross cogross force-pushed the 2321-fix-major-issues-sq branch 3 times, most recently from 6104721 to b56572c Compare May 6, 2024 10:46
@cogross cogross force-pushed the 2321-fix-major-issues-sq branch 5 times, most recently from a20f5dc to 55f7e84 Compare May 13, 2024 15:31
@cogross cogross force-pushed the 2321-fix-major-issues-sq branch 2 times, most recently from ff4f110 to 266af42 Compare May 17, 2024 11:52
@cogross cogross requested a review from jwomeara as a code owner May 21, 2024 17:13
@cogross cogross force-pushed the 2321-fix-major-issues-sq branch 2 times, most recently from 8b5c522 to 0d8bd4c Compare June 3, 2024 10:26
@cogross cogross force-pushed the 2321-fix-major-issues-sq branch 2 times, most recently from d10d93a to 6d4d795 Compare June 10, 2024 15:41
@cogross cogross force-pushed the 2321-fix-major-issues-sq branch 3 times, most recently from fa465c9 to 69b52ff Compare July 17, 2024 16:20
@cogross cogross requested a review from ivakegg as a code owner July 17, 2024 16:20
@cogross cogross force-pushed the 2321-fix-major-issues-sq branch 2 times, most recently from 00f3821 to 33c1585 Compare August 1, 2024 11:45
@cogross cogross force-pushed the 2321-fix-major-issues-sq branch 2 times, most recently from 906f6a7 to b68ae13 Compare August 12, 2024 11:12
@cogross cogross force-pushed the 2321-fix-major-issues-sq branch 3 times, most recently from b9f7060 to 9959030 Compare August 27, 2024 10:55
@cogross cogross force-pushed the 2321-fix-major-issues-sq branch 4 times, most recently from 0ed3c67 to 2916175 Compare September 3, 2024 15:43
@cogross cogross force-pushed the 2321-fix-major-issues-sq branch 4 times, most recently from 04193c5 to b25ce74 Compare September 16, 2024 15:46
@cogross cogross force-pushed the 2321-fix-major-issues-sq branch 3 times, most recently from 974a325 to ca3f7e4 Compare October 2, 2024 18:39
@cogross cogross force-pushed the 2321-fix-major-issues-sq branch 3 times, most recently from 4d83909 to aaa5497 Compare November 4, 2024 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sparkyy34 sparkyy34 sparkyy34 left review comments

@jwomeara jwomeara Awaiting requested review from jwomeara

@ivakegg ivakegg Awaiting requested review from ivakegg

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cogross @sparkyy34