NOAA-EMC / wxflow

Tools for Weather Workflows
https://wxflow.readthedocs.io/en/latest
GNU Lesser General Public License v3.0
0 stars 7 forks source link

Remove empty elements from comma-separated list #40

Closed WalterKolczynski-NOAA closed 1 month ago

WalterKolczynski-NOAA commented 1 month ago

Description

Updates the list parsing in Configuration to remove empty strings from the comma-separated list. This will allow creating a single-element list by appending a , after the element, as well as just relax syntax on lists by ignore a trailing ,.

Refs: NOAA-EMC/global-workflow#2795

Type of change

How Has This Been Tested?

Checklist

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 50.12%. Comparing base (264e8f3) to head (7174ac4).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #40 +/- ## ======================================== Coverage 50.12% 50.12% ======================================== Files 18 18 Lines 1652 1652 Branches 339 339 ======================================== Hits 828 828 Misses 765 765 Partials 59 59 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.