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

validation: Confirm yaml file contents are an object #1084

Merged
merged 1 commit into from
May 15, 2024

Conversation

bjhargrave
Copy link
Contributor

The top-level element of a yaml document can be an object (key-value pairs) or a list. We now test and provide an error message if the top-level element is not an object.

See instructlab/instructlab#1170

The top-level element of a yaml document can be an object (key-value
pairs) or a list. We now test and provide an error message if the
top-level element is not an object.

See instructlab/instructlab#1170

Signed-off-by: BJ Hargrave <[email protected]>
@bjhargrave bjhargrave requested a review from jjasghar May 15, 2024 17:48
@github-actions github-actions bot added the ci label May 15, 2024
@bjhargrave bjhargrave merged commit 3eb556c into instructlab:main May 15, 2024
3 checks passed
@bjhargrave bjhargrave deleted the yaml-list-detection branch May 15, 2024 18:35
dal0643 pushed a commit to dal0643/taxonomy that referenced this pull request May 17, 2024
The top-level element of a yaml document can be an object (key-value
pairs) or a list. We now test and provide an error message if the
top-level element is not an object.

See instructlab/instructlab#1170

Signed-off-by: BJ Hargrave <[email protected]>
Signed-off-by: “David <[email protected]>
jjasghar pushed a commit to luke-inglis/taxonomy that referenced this pull request Jun 7, 2024
The top-level element of a yaml document can be an object (key-value
pairs) or a list. We now test and provide an error message if the
top-level element is not an object.

See instructlab/instructlab#1170

Signed-off-by: BJ Hargrave <[email protected]>
jjasghar pushed a commit to acidonper/taxonomy that referenced this pull request Jun 10, 2024
The top-level element of a yaml document can be an object (key-value
pairs) or a list. We now test and provide an error message if the
top-level element is not an object.

See instructlab/instructlab#1170

Signed-off-by: BJ Hargrave <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants