Skip to content

Conversation

JustinXHale
Copy link
Member

This adds a markdown and Jupyter version of the UX review template. Intended for contributors and reviewers to standardize UX alignment for all notebooks.

@JustinXHale JustinXHale force-pushed the add-ux-review-template branch from e55c736 to 00b55fc Compare April 22, 2025 15:06
@alimaredia alimaredia self-requested a review April 22, 2025 15:20
@alimaredia
Copy link
Contributor

This looks great to me! The only change I would make is I'd only have there be one file, and that file be called:
Notebook-UX-Review-Template.md.

Since we'd just want to have these UX reviews as a comment to PRs, and have it live at the root of the repo right now, not in the templates directory.

@JustinXHale JustinXHale force-pushed the add-ux-review-template branch 6 times, most recently from 237efa5 to 97bccea Compare April 30, 2025 15:59
@JustinXHale JustinXHale added the documentation Improvements or additions to documentation label Apr 30, 2025
justinxhale added 3 commits April 30, 2025 13:37
Signed-off-by: justinxhale <justinxhale@gmail.com>
Signed-off-by: justinxhale <justinxhale@gmail.com>
Signed-off-by: justinxhale <justinxhale@gmail.com>
@JustinXHale JustinXHale force-pushed the add-ux-review-template branch from 74c4312 to c7d6eda Compare April 30, 2025 18:38
@alimaredia alimaredia merged commit 8199513 into instructlab:main Apr 30, 2025
1 check passed
@JustinXHale JustinXHale deleted the add-ux-review-template branch June 5, 2025 20:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants