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

PR: Create Git templates #163

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

nverbic
Copy link
Contributor

@nverbic nverbic commented Dec 10, 2024

Closes #123

Add feature_request.md:

  • template for submitting new features to the GitHub project
  • template for submitting bug
  • template for submitting PR
  • CONTRIBUTING.md

TODO:

  • template for submitting tests
  • maybe template for doc

Add feature_request.md:
 - template for submitting new features to the GitHub project
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Please leave your suggestions and ideas.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Please have a look under the website templates: https://github.com/Women-Coding-Community/WomenCodingCommunity.github.io/tree/main/.github here is where we should create the templates

Copy link
Contributor Author

@nverbic nverbic Dec 11, 2024

Choose a reason for hiding this comment

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

@dricazenck This is the other repository, is it ok?

I thought we wanted to add to this one https://github.com/Women-Coding-Community/wcc-backend/tree/main/.github - but I have no premissions to update the Settings.

Copy link
Contributor

@nora-weisser nora-weisser left a comment

Choose a reason for hiding this comment

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

Great job on creating this template! 👏 I've reviewed it and left a few comments for your consideration. Feel free to apply or decline them based on your judgment.
Also, this template should be created under .github in separate folder called ISSUE_TEMPLATE, so it can be applied automatically when you create an issue in the repo.


### Title

*Example: Create Members GET API*
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need to duplicate the title in the description? The title will be automatically populated with the "feat: " as you put on 4th line of the doc.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, less is more - I will remove this line.


### Description

Description of the new feature and the problem it is solving.
Copy link
Contributor

Choose a reason for hiding this comment

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

Just a suggestion (with more explanation of what to expect in this part).

Provide description of a new feature. Include:  
- How it should work.  
- Any specific requirements or edge cases.  
- Mockups or examples, if applicable.

And I like the example 👍

Comment on lines 30 to 32
*Example: When Client use GET to access members resources that are located at the specified URL on
the
server the server will respond with the list of members and all its data.*
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
*Example: When Client use GET to access members resources that are located at the specified URL on
the
server the server will respond with the list of members and all its data.*
*Example: When Client use GET to access members resources that are located at the specified URL on the server the server will respond with the list of members and all its data.*

should it be formatted like this?

the
server the server will respond with the list of members and all its data.*

### Solution
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
### Solution
### Possible Solution


Describe the solution

*Ex: Use the Controller-Service-Repository pattern from Spring Boot applications. The Controller is
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
*Ex: Use the Controller-Service-Repository pattern from Spring Boot applications. The Controller is
*Example: Use the Controller-Service-Repository pattern from Spring Boot applications. The Controller is

Comment on lines 43 to 45
### Screenhoots

Add screenshots
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
### Screenhoots
Add screenshots
### Additional Context
Include any other context, logs, or screenshots that might help explain your request.

Add PR suggestions
Add bug_report.md:
  - bug_report from the community's repository (link)
Add PULL_REQUEST_TEMPLATE.md:
  - use existing from the community's repository (link)
  - add example
 Add CONTRIBUTING.md
  - use existing from the community's repository (link)
  - update link references to wcc-backend repository
Copy link

sonarcloud bot commented Dec 12, 2024

Copy link
Contributor Author

Choose a reason for hiding this comment

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

File is same as the file here

I have updated some references to wcc-backend repository.

Maybe we can remove or update some descriptios in this file. I would remove this section:

Alternatively, Git CLI way to Sync forked repository with origin repository

and leave section:

Additional way to Sync forked repository with origin repository using Git CLI:

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.

doc: Create Git templates
3 participants