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

Check for wrong names or values in the script_xrefs parameters #650

Merged
merged 3 commits into from
Feb 9, 2021

Conversation

jjnicola
Copy link
Member

@jjnicola jjnicola commented Feb 8, 2021

What:
Check for wrong names or values in the script_xrefs parameters.

Why:
Comma is not allowed and should be checked.

How:
Add a comma in an xrefs in the description block and check it with openvas-nasl-lint

Checklist:

@jjnicola
Copy link
Member Author

jjnicola commented Feb 8, 2021

Please let me know if I should check for other symbols.

Copy link
Member

@cfi-gb cfi-gb left a comment

Choose a reason for hiding this comment

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

There are a few typos found:

$ codespell -e CHANGELOG.md nasl/lint.c 
nasl/lint.c:552: otherwhise ==> otherwise
nasl/lint.c:639: Limite ==> Limit
nasl/lint.c:642: otherwhise ==> otherwise

Code changes needs to be reviewed and approved by a @greenbone/gvm-dev team member.

@jjnicola jjnicola marked this pull request as draft February 9, 2021 07:03
As include files does not have description block.
@jjnicola jjnicola requested a review from cfi-gb February 9, 2021 07:43
@jjnicola jjnicola marked this pull request as ready for review February 9, 2021 07:44
Copy link
Member

@cfi-gb cfi-gb left a comment

Choose a reason for hiding this comment

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

Thanks for fixing the typos, approving them with this review.

@jjnicola jjnicola merged commit ddb55b6 into greenbone:openvas-20.08 Feb 9, 2021
@jjnicola jjnicola deleted the lint-xrefs branch February 9, 2021 11:15
@jjnicola
Copy link
Member Author

jjnicola commented Feb 9, 2021

Thanks for fixing the typos, approving them with this review.

thanks to you for pointing them out

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