Skip to content

Comments

Generate documentation using go templates#1500

Merged
zregvart merged 7 commits intoconforma:mainfrom
zregvart:issue/EC-513
Apr 12, 2024
Merged

Generate documentation using go templates#1500
zregvart merged 7 commits intoconforma:mainfrom
zregvart:issue/EC-513

Conversation

@zregvart
Copy link
Contributor

@zregvart zregvart commented Apr 8, 2024

Reference: EC-513

@codecov
Copy link

codecov bot commented Apr 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.99%. Comparing base (2980742) to head (e209e3e).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1500      +/-   ##
==========================================
+ Coverage   80.46%   86.99%   +6.52%     
==========================================
  Files          67       78      +11     
  Lines        4782     5122     +340     
==========================================
+ Hits         3848     4456     +608     
+ Misses        934      666     -268     
Flag Coverage Δ
acceptance 72.66% <ø> (?)
generative 80.46% <ø> (ø)
integration 80.46% <ø> (ø)
unit 80.46% <ø> (ø)

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

see 42 files with indirect coverage changes

@zregvart zregvart changed the title Remove CLI reference Handlebars template Generate documentation using go templates Apr 8, 2024
@simonbaird
Copy link
Member

Didn't test it, but I looked the commits - seems alright to me.

Copy link
Contributor

@lcarva lcarva left a comment

Choose a reason for hiding this comment

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

Sweet!

* `name` (`string`)
* `namespace` (`string`)
* `qualifiers`(`array`)
**[object<key: string, value: string>]
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this need a space after ** ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It does, good catch


- name: INFO
type: string
description: Include rule titles and descriptions in the output. Set to "false" to disable it.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it should be "false". The value has to be a string.

Create the `asciidoc` package and separate the command line reference
documentation generation into the `asciidoc/cli` package.

Reference: EC-513
This removes the builtin Rego documentation generation using the Antora
extension. This will be replaced by go templating.

Reference: EC-513
Removes the Dependabot configuration for Antora extensions and the use
of NPM in the release GitHub workflow.

Reference: EC-513
@zregvart zregvart merged commit f1fe591 into conforma:main Apr 12, 2024
@zregvart zregvart deleted the issue/EC-513 branch April 12, 2024 10:32
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.

3 participants