-
Notifications
You must be signed in to change notification settings - Fork 219
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
Feature/create more GitHub issue/pr template #518 #650
Closed
jorgelaranjo
wants to merge
7
commits into
svelte-society:main
from
jorgelaranjo:feature/Create-more-Github-issue/PR-template-#518
Closed
Changes from all commits
Commits
Show all changes
7 commits
Select commit
Hold shift + click to select a range
e88f3cd
Created add_update_component.md
jorgelaranjo 4b0ec73
Created request_change.md
jorgelaranjo 77ca4aa
Created delete_component.md
jorgelaranjo 680a24f
Created other_changes.md
jorgelaranjo 4483e85
Create emtpy file to create folder online
jorgelaranjo d93b2ad
new templates and reorg
jorgelaranjo cbfb308
keep old template in current folder
jorgelaranjo File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
## π Feature Request | ||
|
||
<!-- Provide a clear and concise description of the requested feature or change. --> | ||
|
||
## π Description | ||
|
||
<!-- Describe why this change is needed. What problem does it solve? What benefits does it bring? --> | ||
|
||
## π Alternatives Considered | ||
|
||
<!-- Describe any alternative solutions or features you've considered. --> | ||
|
||
## π Additional Context | ||
|
||
<!-- Add any other context or screenshots about the feature request here. --> | ||
|
||
## β Checklist | ||
|
||
- [ ] I have searched the existing issues for similar feature requests. | ||
- [ ] I have provided a clear and concise description of my request. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
## π― Changes | ||
|
||
<!-- Describe the changes made in this PR. Is it a new component, tool, or template? Or is it an update to an existing one? --> | ||
|
||
## π Description | ||
|
||
<!-- Provide a detailed description of the changes. What problem does it solve? How does it improve the existing functionality? --> | ||
|
||
## πΈ Screenshots (if applicable) | ||
|
||
<!-- Include screenshots of the new or updated component/tool/template if applicable. --> | ||
Comment on lines
+9
to
+11
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't know about this section. |
||
|
||
## β Checklist | ||
|
||
- [ ] I have given my PR a descriptive title. | ||
- [ ] I have run `pnpm run lint` locally on my changes. | ||
- [ ] I have added tests that prove my fix is effective or that my feature works. | ||
- [ ] I have added necessary documentation (if appropriate). | ||
|
||
## π Related Issues | ||
|
||
<!-- List any related issues or pull requests here, e.g., "Resolves #123" or "Relates to #456". --> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
@@ -0,0 +1,18 @@ | ||||||
## ποΈ Deletion Request | ||||||
|
||||||
<!-- Describe the component/template/tool that is being deleted and the reasons for its removal. --> | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
||||||
## π Rationale | ||||||
|
||||||
<!-- Provide a detailed rationale for the deletion. Why is this component/template/tool no longer needed? What are the benefits of removing it? --> | ||||||
|
||||||
## β Checklist | ||||||
|
||||||
- [ ] I have given my PR a descriptive title. | ||||||
- [ ] I have run `pnpm run lint` locally on my changes. | ||||||
- [ ] I have verified that the deleted component/template/tool is no longer referenced or used anywhere in the codebase. | ||||||
- [ ] I have updated the documentation (if appropriate) to reflect the deletion. | ||||||
|
||||||
## π Related Issues | ||||||
|
||||||
<!-- List any related issues or pull requests here, e.g., "Resolves #123" or "Relates to #456". --> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
## π― Changes | ||
|
||
<!-- Describe the changes made in this PR. Are they minor fixes, documentation updates, or other types of changes? --> | ||
|
||
## π Description | ||
|
||
<!-- Provide a detailed description of the changes. What problem do they solve? How do they improve the existing functionality? --> | ||
|
||
## β Checklist | ||
|
||
- [ ] I have given my PR a descriptive title. | ||
- [ ] I have run `pnpm run lint` locally on my changes. | ||
- [ ] I have updated the documentation (if appropriate). | ||
|
||
## π Related Issues | ||
|
||
<!-- List any related issues or pull requests here, e.g., "Resolves #123" or "Relates to #456". --> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
## π― Changes | ||
|
||
<!-- What changes are made in this PR? Is it a feature or a package submission? --> | ||
|
||
## β Checklist | ||
|
||
- [ ] I have given my PR a descriptive title | ||
- [ ] I have run `pnpm run lint` locally on my changes |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The distinction between component and tool have been removed.
There are only 6 kind of content (and 2 for external source code):