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

Add a sed utility #23

Closed
wants to merge 2 commits into from

Conversation

DavidHuber-NOAA
Copy link
Collaborator

Description
This creates a utility to build a sed search/replace command and pipe the output to another file.

Type of change

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

How Has This Been Tested?

  • pynorms
  • pytests

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes need updates to the documentation. I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • New and existing tests pass with my changes

@DavidHuber-NOAA DavidHuber-NOAA changed the title Add a sed utility and options to ignore missing files Add a sed utility Apr 11, 2024
Copy link

codecov bot commented Apr 11, 2024

Codecov Report

Attention: Patch coverage is 88.88889% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 49.21%. Comparing base (63db259) to head (fb0c04d).

Files Patch % Lines
src/wxflow/file_utils.py 88.88% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop      #23      +/-   ##
===========================================
+ Coverage    47.99%   49.21%   +1.22%     
===========================================
  Files           18       18              
  Lines         1642     1660      +18     
  Branches       330      334       +4     
===========================================
+ Hits           788      817      +29     
+ Misses         798      787      -11     
  Partials        56       56              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@DavidHuber-NOAA DavidHuber-NOAA deleted the feature/sed branch May 3, 2024 12:24
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