Skip to content
This repository was archived by the owner on Sep 25, 2023. It is now read-only.

Remove commas from "desc," [BUG] #57

Open
karikarshivani opened this issue Jul 31, 2020 · 4 comments
Open

Remove commas from "desc," [BUG] #57

karikarshivani opened this issue Jul 31, 2020 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@karikarshivani
Copy link
Contributor

Describe the bug
Recommendations for the Notes section have a comma after "desc" which cause an error during execution

To Reproduce
Steps to reproduce the behavior:

  1. Add the Notes text from Page 19 in the Overlay SOP to overlay.rb
  2. Execute the overlay

Expected behavior
The overlay is expected to run smoothly

@karikarshivani karikarshivani added the bug Something isn't working label Jul 31, 2020
@aaronlippold
Copy link
Contributor

sub-descriptions should have a comma

@aaronlippold
Copy link
Contributor

aaronlippold commented Aug 10, 2020 via email

@karikarshivani
Copy link
Contributor Author

@aaronlippold, yes, however after the keywords. The example listed in the "To Reproduce" section in my first comment has the following:

desc, 'caveat', 'Not applicable for this CMS ARS 3.1 overlay, since the related security control is not included in
CMS ARS 3.1'

The desc, 'caveat', needs to be changed to desc 'caveat',

@karikarshivani
Copy link
Contributor Author

Screenshot from the Overlay SOP for reference:
image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants