yeatmanlab / roar-dashboard

A dashboard to administer ROAR assessments
https://roar.education
Other
4 stars 4 forks source link

ENH: Add PR template #628

Closed richford closed 4 months ago

richford commented 4 months ago

Proposed changes

This PR adds a PR template that will serve as a starting point for all future PRs. It is intended to be a general purpose PR template for use in ALL ROAR repositories. If it is approved and merged for this repository, I will then add it to all other ROAR repos.

Types of changes

What types of changes does your code introduce? Put an x in the boxes that apply

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

Justification of missing checklist items

This PR does not warrant adding any tests.

github-actions[bot] commented 4 months ago

Visit the preview URL for this PR (updated for commit 9208b89):

https://roar-staging--pr628-enh-add-pr-template-mtnr9rvr.web.app

(expires Thu, 06 Jun 2024 21:11:32 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 2631e9c58fd0104ecbfddd72a62245ddac467460

Zio-4 commented 4 months ago

I think we should add a section for images if applicable.

cypress[bot] commented 4 months ago

Passing run #2210 ↗︎

0 26 0 0 Flakiness 0

Details:

Tests for PR 628 "ENH: Add PR template" from commit "9208b893473bf4e3c616f2b4d7c...
Project: roar-dashboard-e2e Commit: 9208b89347
Status: Passed Duration: 03:28 💡
Started: May 30, 2024 9:10 PM Ended: May 30, 2024 9:14 PM

Review all test suite changes for PR #628 ↗︎

richford commented 4 months ago

I think we should add a section for images if applicable.

Resolved in latest commit by adding a suggestion for images in the HTML comment for the first section.