Skip to content

fix: correctly parse all examples using LINE_PATTERN with named groups#6

Open
dupuy wants to merge 3 commits intooijkn:mainfrom
dupuy:parse-line-with-named-groups
Open

fix: correctly parse all examples using LINE_PATTERN with named groups#6
dupuy wants to merge 3 commits intooijkn:mainfrom
dupuy:parse-line-with-named-groups

Conversation

@dupuy
Copy link
Copy Markdown

@dupuy dupuy commented Mar 28, 2024

A more complete fix for #3:

  • fix: update CSV sample_output file with output from version 1.0.3
    The CSV output from aws-nuke-exporter has CRLF line endings, (that is the default for the Python csv module). Depending on Git configuration, the CR may be removed.
    You can use the --ignore-space-at-eol option to get more useful output from git diff in that case.

  • fix: use correct format for ECSTask resource in sample output text
    Update generated CSV and JSON sample_output files.

  • fix: correctly parse all examples using LINE_PATTERN with named groups
    Update generated CSV and JSON sample_output files.

dupuy added 3 commits March 28, 2024 17:14
The CSV output from aws-nuke-exporter has CRLF line endings,
(that is the default for the Python csv module).
Depending on Git configuration, the CR may be removed.
You can use the --ignore-space-at-eol option to get more useful
output from 'git diff' in that case.
Update generated CSV and JSON sample_output files.
Update generated CSV and JSON sample_output files.
@dupuy dupuy changed the title parse line with named groups fix: correctly parse all examples using LINE_PATTERN with named groups Mar 28, 2024
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.

1 participant