Skip to content
This repository has been archived by the owner on Jun 9, 2024. It is now read-only.

PR/Issue テンプレートを追加しました #1305

Closed
wants to merge 1 commit into from

Conversation

satellitex
Copy link

@satellitex satellitex commented Nov 13, 2023

Summary by CodeRabbit

  • Documentation
    • Introduced a new bug report template to streamline the reporting process with detailed sections for a thorough description.
    • Added a feature request template to guide users through proposing new features effectively.
    • Updated the pull request template with a "What's in this PR" section and a review checklist to improve the quality of submissions.

Copy link

coderabbitai bot commented Nov 13, 2023

Walkthrough

The repository has been updated with new templates to streamline the process of reporting bugs, requesting features, and submitting pull requests. These templates are designed to ensure that contributors provide all the necessary information, which should help maintainers understand and address issues and suggestions more efficiently.

Changes

File Path Change Summary
.github/ISSUE_TEMPLATE/bug_report.md Added a bug report template with detailed sections for reporting issues.
.github/ISSUE_TEMPLATE/feature_request.md Introduced a feature request template to outline new feature suggestions.
.github/PULL_REQUEST_TEMPLATE.md Updated the pull request template with a new section and a review checklist.

🐇 In the fall of GitHub's grace, 🍂
Templates bloom, like stars in space. ✨
Bugs and features, in their place,
CodeRabbit hops at a steady pace. 🚀


Tips

Chat with CodeRabbit Bot (@coderabbitai)

  • If you reply to a review comment from CodeRabbit, the bot will automatically respond.
  • To engage with CodeRabbit bot directly around the specific lines of code in the PR, mention @coderabbitai in your review comment
  • Note: Review comments are made on code diffs or files, not on the PR overview.
  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

CodeRabbit Commands (invoked as PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger a review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai help to get help.

Note: For conversation with the bot, please use the review comments on code diffs or files.

CodeRabbit Configration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • The JSON schema for the configuration file is available here.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/coderabbit-overrides.v2.json

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Review Status

Actionable comments generated: 2

Configuration used: CodeRabbit UI

Commits Files that changed from the base of the PR and between 98379d0 and ac8b7a1.
Files selected for processing (3)
  • .github/ISSUE_TEMPLATE/bug_report.md (1 hunks)
  • .github/ISSUE_TEMPLATE/feature_request.md (1 hunks)
  • .github/PULL_REQUEST_TEMPLATE.md (1 hunks)
Files skipped from review due to trivial changes (1)
  • .github/ISSUE_TEMPLATE/feature_request.md
Additional comments: 1
.github/ISSUE_TEMPLATE/bug_report.md (1)
  • 2-7: The metadata fields are in Japanese. Verify that this is intentional and that the target audience for this repository includes Japanese speakers. If not, these fields should be translated into English to maintain consistency with the rest of the template.

Comment on lines +1 to +6
## What's PR


## Review checklist
- [ ] xx
- [ ] oo
Copy link

Choose a reason for hiding this comment

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

The new section "What's PR" should have a brief explanation or instruction on what should be included in this section. The review checklist items 'xx' and 'oo' are placeholders and should be replaced with actual criteria that need to be checked before a pull request is considered ready for review or merging.

## What's PR
+ Please provide a summary of the pull request's purpose and any key changes made.


## Review checklist
- [ ] xx
+ [ ] Code complies with the project's coding standards
- [ ] oo
+ [ ] All tests pass

Commitable suggestion

[!IMPORTANT]
Carefully review the code before committing. Make sure it correctly replaces the highlighted code, has no missing lines and indentaion issues.

Suggested change
## What's PR
## Review checklist
- [ ] xx
- [ ] oo
## What's PR
Please provide a summary of the pull request's purpose and any key changes made.
## Review checklist
- [ ] Code complies with the project's coding standards
- [ ] All tests pass

Comment on lines +9 to +40
**バグの説明**
そのバグが何であるかを明確かつ簡潔に記述してください。

\*\*再現方法
その動作を再現するための手順。

1. '...' に移動します。
2. '...' をクリックします。
3. スクロールダウンして '......' をクリックします。
4. エラーを見る

\*\*期待される動作
期待した動作の明確で簡潔な説明。

**スクリーンショット**
該当する場合、問題を説明するためにスクリーンショットを追加してください。

**デスクトップ(以下の情報を記入してください):**。

- OS。[例:iOS](アイオーエス
- ブラウザ [例:クローム、サファリ]。
- バージョン [例: 22]

\*\*スマートフォン(以下の項目を入力してください)

- デバイス [例:iPhone6]
- OS: [例:iOS8.1]
- ブラウザ[例:純正ブラウザ、safari]。
- バージョン [例:22]。

**その他の状況**について
問題についてのその他の状況をここに追加してください。
Copy link

Choose a reason for hiding this comment

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

The bug report template is well-structured and covers all the necessary sections to gather detailed information about a bug. However, the language used in the template is mixed, with some parts in Japanese and others in English. Ensure that the language is consistent throughout the document to avoid confusion. If the repository is intended for a multilingual audience, consider providing separate templates for each language or including language-specific sections within the same template.

@satellitex satellitex closed this Nov 13, 2023
@peaceandwhisky peaceandwhisky deleted the feat/pr-template branch November 29, 2023 02:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant