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

Update Pull Request template #614

Merged
merged 3 commits into from
Oct 29, 2022

Conversation

muzaffar640
Copy link
Contributor

@muzaffar640 muzaffar640 commented Oct 5, 2022

First thing, PLEASE READ THIS: ReactPlay Code Review Checklist

Description

Added Screebshot or example output block to Pull Request Template

Fixes # 599

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Checked the output on VS Code using command+shift+v

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

Screenshots or example output

Screenshot 2022-10-06 at 12 46 27 AM

@vercel
Copy link

vercel bot commented Oct 5, 2022

@muzaffar640 is attempting to deploy a commit to a Personal Account owned by @reactplay on Vercel.

@reactplay first needs to authorize it.

@muzaffar640 muzaffar640 marked this pull request as ready for review October 5, 2022 19:20
@muzaffar640
Copy link
Contributor Author

Review Ready

@atapas atapas linked an issue Oct 6, 2022 that may be closed by this pull request
@vercel
Copy link

vercel bot commented Oct 6, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
react-play ✅ Ready (Inspect) Visit Preview Oct 6, 2022 at 4:44AM (UTC)

Copy link
Member

@atapas atapas left a comment

Choose a reason for hiding this comment

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

@muzaffar640 This is good but I am thinking, can we go yml way so that it becomes more like form filling for the contributors? Are you aware of that template creation?

@muzaffar640
Copy link
Contributor Author

@muzaffar640 This is good but I am thinking, can we go yml way so that it becomes more like form filling for the contributors? Are you aware of that template creation?

No @atapas, I'm not aware of that. Can share some references, please?

@atapas
Copy link
Member

atapas commented Oct 7, 2022

@muzaffar640 This is good but I am thinking, can we go yml way so that it becomes more like form filling for the contributors? Are you aware of that template creation?

No @atapas, I'm not aware of that. Can share some references, please?

Like this: https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/syntax-for-githubs-form-schema

@muzaffar640
Copy link
Contributor Author

@atapas I am playing around with it. From the documentation, I'm able to create various issue templates in yml way. And as you said it is looking really good 😁 . But for PR template, I have tried various methods to render the yml file but could not able to succeed. And didn't find any valid documentation also. Can you please help me with this one? Here is my test repo

I want to update the issue template also if suggest. Please have a look at this.

Screen.Recording.2022-10-08.at.1.27.13.PM.mov

@atapas
Copy link
Member

atapas commented Oct 8, 2022

@atapas I am playing around with it. From the documentation, I'm able to create various issue templates in yml way. And as you said it is looking really good 😁 . But for PR template, I have tried various methods to render the yml file but could not able to succeed. And didn't find any valid documentation also. Can you please help me with this one? Here is my test repo

I want to update the issue template also if suggest. Please have a look at this.

Screen.Recording.2022-10-08.at.1.27.13.PM.mov

It seems this support is for the issue alone. I also tried the PR template but not successful. Let us then do these.

  • Will go ahead with your suggested changes for PR template you made originally.
  • Change the issue templates.

We will continue the PR template using YAML in a separate issue. Does it sound good?

@muzaffar640
Copy link
Contributor Author

@atapas I am playing around with it. From the documentation, I'm able to create various issue templates in yml way. And as you said it is looking really good 😁 . But for PR template, I have tried various methods to render the yml file but could not able to succeed. And didn't find any valid documentation also. Can you please help me with this one? Here is my test repo
I want to update the issue template also if suggest. Please have a look at this.
Screen.Recording.2022-10-08.at.1.27.13.PM.mov

It seems this support is for the issue alone. I also tried the PR template but not successful. Let us then do these.

  • Will go ahead with your suggested changes for PR template you made originally.
  • Change the issue templates.

We will continue the PR template using YAML in a separate issue. Does it sound good?

That will be better. I will create a new issue and PR for Create Issue Template with yml then. Thanks for the confirmation.

@koustov
Copy link
Member

koustov commented Oct 19, 2022

@atapas is it ready to go in?

@atapas atapas added the atapas label Oct 29, 2022
@atapas atapas added the hacktoberfest-accepted hacktoberfest-accepted label Oct 29, 2022
@atapas atapas merged commit a462652 into reactplay:main Oct 29, 2022
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.

✨ [Feature request]: Improve Pull Request Template
4 participants