Skip to content

[WIP] [WIP] Address feedback on yaml validation and decoding in parameters handling#151

Closed
Copilot wants to merge 1 commit intolarryo/validate_yamlfrom
copilot/sub-pr-133-another-one
Closed

[WIP] [WIP] Address feedback on yaml validation and decoding in parameters handling#151
Copilot wants to merge 1 commit intolarryo/validate_yamlfrom
copilot/sub-pr-133-another-one

Conversation

Copy link
Contributor

Copilot AI commented Mar 19, 2026

Thanks for the feedback on #133. I've created this new PR, which merges into #133, to address your comment. I will work on the changes and keep this PR's description up to date as I make progress.

Original PR: #133
Triggering review: #133 (comment)

@copilot decodeGraderParameters (which does the actual parsing of grader parameters) uses a strict yaml.Decoder for each of the grader parameters - it serializes the yaml.Node into a bytes array and then decodes it using a struct decoder. Isn't that sufficient?


⚡ Quickly spin up Copilot coding agent tasks from anywhere on your macOS or Windows machine with Raycast.

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.

2 participants