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

fix: code generation doesn't ignore dotfiles #348

Merged
merged 4 commits into from May 16, 2022

Conversation

katcipis
Copy link
Contributor

Currently code generation will read dotfiles, but those should be ignored.

@katcipis katcipis requested a review from i4ki May 13, 2022 14:57
@katcipis katcipis self-assigned this May 13, 2022
@codecov
Copy link

codecov bot commented May 13, 2022

Codecov Report

Merging #348 (1adb846) into main (c9a486b) will increase coverage by 0.06%.
The diff coverage is 80.32%.

@@            Coverage Diff             @@
##             main     #348      +/-   ##
==========================================
+ Coverage   67.73%   67.79%   +0.06%     
==========================================
  Files          36       36              
  Lines        6670     6683      +13     
==========================================
+ Hits         4518     4531      +13     
  Misses       1902     1902              
  Partials      250      250              
Flag Coverage Δ
tests 67.79% <80.32%> (+0.06%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
hcl/hcl.go 85.78% <80.32%> (+0.23%) ⬆️

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 c9a486b...1adb846. Read the comment docs.

Copy link
Contributor

@i4ki i4ki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@i4ki
Copy link
Contributor

i4ki commented May 16, 2022

sorry: we need to get rid of the codegen/hcl duplication ASAP.

@katcipis
Copy link
Contributor Author

sorry: we need to get rid of the codegen/hcl duplication ASAP.

Yeah lets introduce some improvements on this soon 🚀

@katcipis katcipis merged commit cd89f43 into main May 16, 2022
@katcipis katcipis deleted the katcipis-fix-generate-ignore-dotfiles branch May 16, 2022 17:07
@katcipis katcipis changed the title fix: code generation does't ignore dotfiles fix: code generation doesn't ignore dotfiles Jun 1, 2022
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.

None yet

2 participants