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 API for use as -X program or -T CLI tool #10

Merged
merged 22 commits into from
Mar 16, 2022
Merged

Add API for use as -X program or -T CLI tool #10

merged 22 commits into from
Mar 16, 2022

Conversation

athos
Copy link
Owner

@athos athos commented Feb 20, 2022

It would be useful to be able to call some of the functions from the CLI via the -X/-T options.

This PR exposes the following two features as such an API:

  • render: Renders the given template
  • check: Checks one or more templates

@codecov
Copy link

codecov bot commented Feb 21, 2022

Codecov Report

Merging #10 (685159a) into master (8190b17) will decrease coverage by 0.18%.
The diff coverage is 84.81%.

@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
- Coverage   90.56%   90.37%   -0.19%     
==========================================
  Files          10       11       +1     
  Lines         742      821      +79     
  Branches       37       44       +7     
==========================================
+ Hits          672      742      +70     
- Misses         33       35       +2     
- Partials       37       44       +7     
Impacted Files Coverage Δ
src/pogonos/api.clj 84.81% <84.81%> (ø)
src/pogonos/output.cljc 87.50% <0.00%> (+3.12%) ⬆️
src/pogonos/error.cljc 88.37% <0.00%> (+4.65%) ⬆️

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 8190b17...685159a. Read the comment docs.

@athos
Copy link
Owner Author

athos commented Mar 16, 2022

There is not yet sufficient documentation for this change, but I think it's better to merge the change first than to stall here.

@athos athos marked this pull request as ready for review March 16, 2022 14:29
@athos athos merged commit 3125449 into master Mar 16, 2022
@athos athos deleted the feature/api branch March 16, 2022 14:30
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