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

Create PULL_REQUEST_TEMPLATE.md #785

Merged
merged 4 commits into from Oct 24, 2021
Merged

Create PULL_REQUEST_TEMPLATE.md #785

merged 4 commits into from Oct 24, 2021

Conversation

Saibamen
Copy link
Contributor

@Saibamen Saibamen commented Oct 24, 2021

Changes after Code Review
@louislam louislam merged commit 5d0b54c into louislam:master Oct 24, 2021
@Saibamen Saibamen deleted the patch-1 branch October 24, 2021 19:01
@louislam
Copy link
Owner

louislam commented Oct 25, 2021

I found that using checkbox may not be a good approach for "Type of change", since it look like that the progress cannot be done in the list view.

image

@Saibamen
Copy link
Contributor Author

@louislam: Change it to list?
Both Type of change and Checklist?

@louislam
Copy link
Owner

It is good to change Type of change into a list.

For Checklist, I think "My code needed automated testing. I have added them" is optional.

Saibamen added a commit to Saibamen/uptime-kuma that referenced this pull request Oct 26, 2021
@Saibamen Saibamen mentioned this pull request Oct 26, 2021
6 tasks
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.

None yet

3 participants