CMSgov/saf

Remove commas from "desc," [BUG]

Opened this issue · 4 comments

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

sub-descriptions should have a comma

@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',

Screenshot from the Overlay SOP for reference:
image