Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix checks with comma issues in checks @j2clerck #975

Merged
merged 2 commits into from
Dec 22, 2021
Merged

Fix checks with comma issues in checks @j2clerck #975

merged 2 commits into from
Dec 22, 2021

Conversation

j2clerck
Copy link

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

checks/check122 Show resolved Hide resolved
checks/check_extra7149 Show resolved Hide resolved
@toniblyx toniblyx changed the title fix checks with comma issues Fix checks with comma issues in checks Dec 21, 2021
@toniblyx toniblyx changed the title Fix checks with comma issues in checks Fix checks with comma issues in checks @j2clerck Dec 21, 2021
@toniblyx
Copy link
Member

As mentioned in Discord, running grep \, checks/*| grep -E "textFail|textPass|textInfo|RISK|REMEDIATION" I get extra7152, extra7153, extra7156, extra7162, extra765, extra769 and extra794 that have to be changed. Let me know if you can make it or I can do it.

@j2clerck
Copy link
Author

j2clerck commented Dec 21, 2021

@toniblyx I will have a look at the remaining checks, but probably later in the day (Eastern Time)

@toniblyx
Copy link
Member

Awesome, thanks @j2clerck

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants