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 plain Terraform support via "plugable" IncludeFn() #32

Merged
merged 6 commits into from
Nov 15, 2022
Merged

Conversation

ivanilves
Copy link
Owner

@ivanilves ivanilves commented Nov 14, 2022

Fixes #23

Pass a custom "include" function to get flexible and easily configurable rules.

Added for Terraform, but should work for any file-based projects.

💡 In future config should be extended with configurable rules not sitting directly in code, but expressed in YAML.

@codecov
Copy link

codecov bot commented Nov 14, 2022

Codecov Report

Merging #32 (12d39f7) into main (4724eda) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #32   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         5    +2     
  Lines          163       200   +37     
=========================================
+ Hits           163       200   +37     
Impacted Files Coverage Δ
pkg/config/config.go 100.00% <100.00%> (ø)
pkg/config/include/include.go 100.00% <100.00%> (ø)
pkg/directory/directory.go 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@ivanilves ivanilves changed the title Add plain Terraform support via "plugable" includes Add plain Terraform support via "plugable" IncludeFn() Nov 15, 2022
Copy link
Collaborator

@roderic1993 roderic1993 left a comment

Choose a reason for hiding this comment

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

please add documentation

@ivanilves ivanilves merged commit 8f95996 into main Nov 15, 2022
@ivanilves ivanilves deleted the GH-23 branch November 15, 2022 04:18
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.

Add plain Terraform support
2 participants