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

Yaml Validator Iteration One into Main Branch #677

Merged
merged 12 commits into from
Jun 25, 2024

Conversation

abaskk-msft
Copy link
Contributor

@abaskk-msft abaskk-msft commented Jun 24, 2024

Feature

  • This feature adds a Yaml Validator under the Persistence namespace. The inputs accepted are either a single yaml file or a directory of yaml files. Here is a rough UML of the structure of the project.
  • Additionally users are able to see a stack trace of syntax errors. Specifically the file where an error was found and where the exact error with yaml pointer and schema pointer paths.
image

Validation

  • Tested the VSCode extension still works despite the changes in schema format
  • Unit tested the Validator class

@abaskk-msft abaskk-msft requested review from a team as code owners June 24, 2024 20:14
src/YAMLValidator.Tests/ValidatorTest.cs Outdated Show resolved Hide resolved
src/YAMLValidator/InputProcessor.cs Outdated Show resolved Hide resolved
@petrochuk petrochuk self-requested a review June 24, 2024 23:17
docs/pa.yaml-schema.json Outdated Show resolved Hide resolved
src/YAMLValidator/Constants.cs Outdated Show resolved Hide resolved
src/YAMLValidator/Utility.cs Outdated Show resolved Hide resolved
src/YAMLValidator/YAMLValidator.csproj Outdated Show resolved Hide resolved
@abaskk-msft abaskk-msft merged commit c57c3d4 into master Jun 25, 2024
4 checks passed
@abaskk-msft abaskk-msft deleted the users/t-abaskar/yaml-validator branch June 25, 2024 17:44
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.

4 participants