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

Issue No. #177 [FIXED] Pull Request Template Added. #191

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

gauravsingh1281
Copy link
Contributor

@gauravsingh1281 gauravsingh1281 commented Jan 31, 2025

Closes #177

🚀 Pull Request

Description
A clear and concise description of the changes introduced in this pull request.


Related Issue
If this pull request addresses an issue, link to it here (e.g., Closes #123).


Type of Change
What kind of change does this pull request introduce?

  • 🐛 Bug fix
  • 🚀 New feature
  • 📄 Documentation update
  • 🛠️ Code improvement/refactor
  • 🖼️ UI/UX update
  • Other (please specify)

Screenshots (if applicable)
Add screenshots or screen recordings to demonstrate the new feature or fix.


Checklist
Please ensure the following tasks are complete before submitting the pull request:

  • I have tested my changes and ensured they work as expected.
  • I have added any necessary documentation or comments.
  • I have checked the existing issues and pull requests to avoid duplicates.
  • I have starred this repository.
  • I have reviewed my code for best practices and readability.
  • I have included any relevant tests (if applicable).

Additional Context
Add any other relevant information or context about the pull request here.

@Aarzookhunger @Mayureshd-18 @Abhay-hack Please add under intermediate level.

Copy link

vercel bot commented Jan 31, 2025

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

Name Status Preview Comments Updated (UTC)
lumina ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 1, 2025 5:05pm

@Aarzookhunger
Copy link
Collaborator

Closes #177

🚀 Pull Request

Description A clear and concise description of the changes introduced in this pull request.

Related Issue If this pull request addresses an issue, link to it here (e.g., Closes #123).

Type of Change What kind of change does this pull request introduce?

  • 🐛 Bug fix
  • 🚀 New feature
  • 📄 Documentation update
  • 🛠️ Code improvement/refactor
  • 🖼️ UI/UX update
  • Other (please specify)

Screenshots (if applicable) Add screenshots or screen recordings to demonstrate the new feature or fix.

Checklist Please ensure the following tasks are complete before submitting the pull request:

  • I have tested my changes and ensured they work as expected.
  • I have added any necessary documentation or comments.
  • I have checked the existing issues and pull requests to avoid duplicates.
  • I have starred this repository.
  • I have reviewed my code for best practices and readability.
  • I have included any relevant tests (if applicable).

Additional Context Add any other relevant information or context about the pull request here.

@Aarzookhunger @Mayureshd-18 @Abhay-hack Please add under intermediate level.

@gauravsingh1281 we'll assign the label as per the work done, kindly don't mention under each PR/issue for assigning a certain label.Thankyou!

Copy link
Collaborator

@Mayureshd-18 Mayureshd-18 left a comment

Choose a reason for hiding this comment

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

@gauravsingh1281 Please have a working example in your forked repo as an example.

For any workflow related PRs its important to test the workflows in the forked repo.

Thanks and regards,
Mayuresh

@Mayureshd-18
Copy link
Collaborator

@gauravsingh1281 I am not abe to see the template after I tried creating a PR in ur repo.

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.

[Feature] Add Pull Request Template.
3 participants