Skip to content
This repository has been archived by the owner on Nov 4, 2024. It is now read-only.

fix bug: issue #148 #149

Merged
merged 2 commits into from
Sep 14, 2023
Merged

fix bug: issue #148 #149

merged 2 commits into from
Sep 14, 2023

Conversation

GoZaddy
Copy link

@GoZaddy GoZaddy commented Sep 13, 2023

No description provided.

@GoZaddy GoZaddy self-assigned this Sep 13, 2023
Copy link
Member

@ncovercash ncovercash left a comment

Choose a reason for hiding this comment

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

Nice find!

@nhanaa
Copy link

nhanaa commented Sep 14, 2023

@ncovercash The github action somehow fails but I'm not sure what the problem is. Is this concerning?

@ncovercash
Copy link
Member

@Nhanna I can't see it on my phone, but I'm guessing the unit test for this is failing (since it probably passed in the previous (incorrect) property). I'd go check and update the corresponding unit test

@GoZaddy
Copy link
Author

GoZaddy commented Sep 14, 2023

@Nhanna I can't see it on my phone, but I'm guessing the unit test for this is failing (since it probably passed in the previous (incorrect) property). I'd go check and update the corresponding unit test

Oh yeah, my bad. I'll go update that

Copy link
Member

@ncovercash ncovercash left a comment

Choose a reason for hiding this comment

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

lgtm, assuming this fixes test issue

@sonarcloud
Copy link

sonarcloud bot commented Sep 14, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@GoZaddy GoZaddy merged commit f7f07db into master Sep 14, 2023
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants