Skip to content

Include hack (just a quick demonstration, not meant for merge)#23

Open
mzpqnxow wants to merge 2 commits intozobayer1:mainfrom
mzpqnxow:include-hack
Open

Include hack (just a quick demonstration, not meant for merge)#23
mzpqnxow wants to merge 2 commits intozobayer1:mainfrom
mzpqnxow:include-hack

Conversation

@mzpqnxow
Copy link
Copy Markdown
Contributor

Here's a quick pass at supporting !include. It's just a proof of concept that works for very basic !include directives

When !include is encountered, it will do one of the following, based on the extension of the file specified:

  1. If .yml/.yaml: load as a YaML file, producing structured data
  2. Else, consider to be a string and load as such, removing newlines

This is in references to the discussion in #21

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 29, 2022

Codecov Report

Merging #23 (b811925) into main (014bce9) will decrease coverage by 26.31%.
The diff coverage is 37.50%.

Impacted file tree graph

@@             Coverage Diff              @@
##              main      #23       +/-   ##
============================================
- Coverage   100.00%   73.68%   -26.32%     
============================================
  Files            5        5               
  Lines           62       95       +33     
============================================
+ Hits            62       70        +8     
- Misses           0       25       +25     
Impacted Files Coverage Δ
logging_/config/yaml_config.py 64.78% <37.50%> (-35.22%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 014bce9...b811925. Read the comment docs.

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