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 #692 Add the Github Issue Template Form #729

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

000xs
Copy link

@000xs 000xs commented Oct 18, 2024

Resolves #692 : Created New Github Issue Template Form


Important

Add new GitHub issue templates for bug reports, documentation, feature requests, and discussions, and remove old markdown templates.

  • New Templates:
    • Adds bug-report.yml for bug reports with fields for issue description, expected behavior, reproduction steps, and more.
    • Adds documentation.yml for documentation issues with fields for issue description and proposed solutions.
    • Adds feature-request.yml for feature requests with fields for description, proposed solution, and impact.
    • Adds other.yml for general questions or discussions.
  • Removed Templates:
    • Deletes bug_report_frontend.md, bug_report_sdk.md, custom.md, and feature_request.md markdown templates.

This description was created by Ellipsis for 460a0fc. It will automatically update as commits are pushed.

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to 460a0fc in 16 seconds

More details
  • Looked at 480 lines of code in 9 files
  • Skipped 0 files when reviewing.
  • Skipped posting 3 drafted comments based on config settings.
1. .github/ISSUE_TEMPLATE/bug-report.yml:1
  • Draft comment:
    The name and title formatting in this file is inconsistent with the other bug report template (bug_report.yml). Consider standardizing the naming and title format across all templates for consistency.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The bug report templates have inconsistent naming and formatting, which could lead to confusion.
2. .github/ISSUE_TEMPLATE/documentation.yml:37
  • Draft comment:
    The description contains a duplicated link to the Contributing Guidelines. Remove the redundant link to avoid confusion.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The documentation issue template has a duplicated link in the description, which is redundant.
3. .github/ISSUE_TEMPLATE/feature-request.yml:53
  • Draft comment:
    There's a typo in the label description: [Code of Conduct](https://github.com/ComposioHQ/composio/blob/main/CODE_OF_ CONDUCT.md). Remove the space in CODE_OF_ CONDUCT.md.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The feature request template has a typo in the label description, which could lead to confusion.

Workflow ID: wflow_ZWompXZFE9jl1E0P


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@plxity
Copy link
Contributor

plxity commented Dec 11, 2024

@000xs Thanks for working on this. Can you please record a video of this and share, just want to see if everything looks good visually.

Copy link
Collaborator

@tushar-composio tushar-composio left a comment

Choose a reason for hiding this comment

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

Double check all your changes and send a photo of these forms once fixed.

@@ -0,0 +1,119 @@
name: " Bug Report"
Copy link
Collaborator

Choose a reason for hiding this comment

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

typo? also there's two bug report forms

- type: textarea
id: something_wrong
attributes:
label: "What went wrong? "
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove space

id: terms_checklist_bug
attributes:
label: "Code of Conduct"
description: By submitting this issue, you agree to follow our [Code of Conduct](link_to_your_code_of_conduct) and [Contributing Guidelines](link_to_your_contributing_guidelines) (if applicable).
Copy link
Collaborator

Choose a reason for hiding this comment

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

fix the URLs

id: terms_checklist_bug
attributes:
label: "Code of Conduct"
description: By submitting this issue, you agree to follow our [Code of Conduct](https://github.com/ComposioHQ/composio/blob/main/CODE_OF_CONDUCT.md) and [Contributing Guidelines](https://github.com/ComposioHQ/composio/blob/main/CONTRIBUTING.md) and [Contributing Guidelines](link_to_your_contributing_guidelines) (if applicable).
Copy link
Collaborator

Choose a reason for hiding this comment

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

these URLs go to 404

@@ -0,0 +1,62 @@
name: " Feature Request"
description: "Have any new ideas or features for Composio? Please suggest!"
title: "<write a small description here>"
Copy link
Collaborator

Choose a reason for hiding this comment

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

titles should not be used like that

Copy link

vercel bot commented Dec 24, 2024

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

Name Status Preview Comments Updated (UTC)
composio ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 24, 2024 1:44pm

@@ -0,0 +1,119 @@
name: " Bug Report"
Copy link
Collaborator

Choose a reason for hiding this comment

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

There are two bug report template files being added: bug-report.yml and bug_report.yml. These files have similar content but slight differences. This will cause confusion for users. Please consolidate these into a single bug report template.

@@ -0,0 +1,119 @@
name: " Bug Report"
description: I would like to report a bug.
title: "[ BUG] <write a small description here>"
Copy link
Collaborator

Choose a reason for hiding this comment

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

The title format has inconsistent spacing "[ BUG]". For consistency, it should be "[BUG]" without extra spaces.

id: terms_checklist_bug
attributes:
label: "Code of Conduct"
description: By submitting this issue, you agree to follow our [Code of Conduct](link_to_your_code_of_conduct) and [Contributing Guidelines](link_to_your_contributing_guidelines) (if applicable).
Copy link
Collaborator

Choose a reason for hiding this comment

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

The Code of Conduct and Contributing Guidelines links are using placeholder text (link_to_your_code_of_conduct). Please replace these with the actual links to the repository's documents:

  • Code of Conduct: https://github.com/ComposioHQ/composio/blob/main/CODE_OF_CONDUCT.md
  • Contributing Guidelines: https://github.com/ComposioHQ/composio/blob/main/CONTRIBUTING.md

- Brave
- Other

- type: input # Assuming Composio works with different OS
Copy link
Collaborator

Choose a reason for hiding this comment

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

The input type is used here for OS selection, but since you have predefined options, you should use dropdown type instead, similar to how the browser selection is implemented above. This will provide a better user experience and prevent invalid inputs.

label: "Code of Conduct"
description: By submitting this issue, you agree to follow our [Code of Conduct](link_to_your_code_of_conduct) and [Contributing Guidelines](link_to_your_contributing_guidelines) (if applicable).
options:
- label: "I
Copy link
Collaborator

Choose a reason for hiding this comment

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

The file appears to be cut off mid-sentence at line 119 with "I". Please complete the content and ensure all checkboxes are properly defined.

@shreysingla11
Copy link
Collaborator

Code Review Summary

Overall, this PR makes a good improvement by moving from markdown-based templates to more structured YAML-based forms. However, there are several issues that need to be addressed:

Critical Issues:

  1. Duplicate bug report templates need to be consolidated
  2. Incomplete content in bug_report.yml (cut off at line 119)
  3. Placeholder links need to be replaced with actual repository links

Improvements Needed:

  1. Use consistent formatting for titles and labels across templates
  2. Change input type to dropdown for OS selection
  3. Consider adding character limits to text inputs
  4. Add validation for version numbers
  5. Consider adding templates for security vulnerabilities and performance issues

Positive Aspects:

✅ Good organization of different template types
✅ Comprehensive fields for bug reports
✅ Clear instructions and placeholders
✅ Proper use of required/optional fields

Please address the critical issues and consider the suggested improvements. Once these are fixed, the PR will be ready for merge.

Code Quality Rating: 7/10 (Good foundation but needs refinement)

@000xs
Copy link
Author

000xs commented Dec 24, 2024

Code Review Summary

Overall, this PR makes a good improvement by moving from markdown-based templates to more structured YAML-based forms. However, there are several issues that need to be addressed:

Critical Issues:

  1. Duplicate bug report templates need to be consolidated
  2. Incomplete content in bug_report.yml (cut off at line 119)
  3. Placeholder links need to be replaced with actual repository links

Improvements Needed:

  1. Use consistent formatting for titles and labels across templates
  2. Change input type to dropdown for OS selection
  3. Consider adding character limits to text inputs
  4. Add validation for version numbers
  5. Consider adding templates for security vulnerabilities and performance issues

Positive Aspects:

✅ Good organization of different template types ✅ Comprehensive fields for bug reports ✅ Clear instructions and placeholders ✅ Proper use of required/optional fields

Please address the critical issues and consider the suggested improvements. Once these are fixed, the PR will be ready for merge.

Code Quality Rating: 7/10 (Good foundation but needs refinement)

I fixed the typo. Please check if I missed anything.

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.

Add the github issue template form
5 participants