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 commands to validate qlpack.yml and YAML MaD definitions #8

Closed
wants to merge 8 commits into from

Conversation

jeongsoolee09
Copy link
Collaborator

@jeongsoolee09 jeongsoolee09 commented Oct 11, 2023

qlt pack validate syntax; qlt pack validate version

Deps:

dotnet add package YamlDotNet --version 13.7.0
dotnet add package Semver --version 2.3.0

Sketch of the hypothetical usage scenarios:

$ qlt pack validate version --yml src/qlpack.yml test/qlpack.yml ext/qlpack.yml
Checking if the versions match up across the qlpack definitions.
src/qlpack.yml (advanced-security/javascript-sap-ui5-queries) is declared to be v0.3.0.
src/qlpack.yml (advanced-security/javascript-sap-ui5-queries) depends on advanced-security/javascript-sap-ui5-models: [{lb}, {ub}]
src/qlpack.yml (advanced-security/javascript-sap-ui5-queries) depends on advanced-security/javascript-sap-ui5-models: [{lb}, {ub}]
...
ext/qlpack.yml
...
{No contradictions.|{file1}, {file2}, {file3} does not match on pack {dep-pack-name}.}
$ qlt pack validate syntax --ext ext/ui5-extensions.yml
PASS: [some, row, "that's", correct]
FAIL: [some, row, "that's", wrong]
...
Done scanning.
========================================
Here are the rows that needs fixup:
2:4:32: [some, row, "that's", wrong]
17:4:32: [some, row, "that's", wrong]
...

`qlt pack validate syntax`; `qlt pack validate version`
@jeongsoolee09 jeongsoolee09 self-assigned this Oct 11, 2023
@jeongsoolee09
Copy link
Collaborator Author

Closing due to branch being stale.

@jeongsoolee09 jeongsoolee09 deleted the jeongsoolee09/sap-validate-version-syntax branch March 8, 2024 21:56
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