Skip to content

Conversation

bjhargrave
Copy link
Contributor

Describe the contribution to the taxonomy

Rename pattern to patterns in knowledge schema.

See instructlab/schema#3 (comment)

@bjhargrave bjhargrave requested review from a team as code owners April 3, 2024 14:57
@github-actions github-actions bot added triage-needed (Auto labeled) skill is ready to be triaged knowledge (Auto labeled) labels Apr 3, 2024
@bjhargrave
Copy link
Contributor Author

The schema validation failures are due to other issues in the knowledge yaml files (empty description, not enough examples).

Copy link
Contributor

@juliadenham juliadenham left a comment

Choose a reason for hiding this comment

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

LGTM!

@bjhargrave
Copy link
Contributor Author

@xukai92 please review/approve.

@bjhargrave bjhargrave removed the triage-needed (Auto labeled) skill is ready to be triaged label Apr 3, 2024
Copy link
Member

@xukai92 xukai92 left a comment

Choose a reason for hiding this comment

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

LGTM

@xukai92 xukai92 merged commit cf9f3ed into instructlab:main Apr 4, 2024
@xukai92 xukai92 deleted the knowledge-patterns branch April 4, 2024 02:54
xukai92 pushed a commit to instructlab/instructlab that referenced this pull request Apr 4, 2024
See instructlab/taxonomy#654

Signed-off-by: Anik Bhattacharjee <anbhatta@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
knowledge (Auto labeled)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants