Skip to content

refactor: update feature request template - #50

Merged
Ki-Seki merged 1 commit into
MemTensor:devfrom
Ki-Seki:issue-tmplate
Jul 10, 2025
Merged

refactor: update feature request template#50
Ki-Seki merged 1 commit into
MemTensor:devfrom
Ki-Seki:issue-tmplate

Conversation

@Ki-Seki

@Ki-Seki Ki-Seki commented Jul 9, 2025

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings July 9, 2025 14:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the feature request issue template to improve clarity and enforce pre-submission checks.

  • Renames and restructures the reminder checklist with specific required items
  • Updates the description field to a “Problem Statement” with placeholder guidance
  • Changes the contribution prompt from a freeform textarea to predefined checkbox options
Comments suppressed due to low confidence (4)

.github/ISSUE_TEMPLATE/feature-request.yml:15

  • The required: true on the last checklist option appears misaligned; this may break YAML parsing. Align it with the other required entries under options.
          required: true

.github/ISSUE_TEMPLATE/feature-request.yml:33

  • [nitpick] The required: false flag is redundant since unchecked is the default behavior; consider removing it for brevity.
          required: false

.github/ISSUE_TEMPLATE/feature-request.yml:18

  • [nitpick] The id value problem is somewhat generic; consider renaming to problem_statement to match the label and improve clarity.
    id: problem

.github/ISSUE_TEMPLATE/feature-request.yml:8

  • There is no description explaining the purpose of this checklist; adding a description attribute could help users understand why these checks are required.
      label: Pre-submission checklist

@Ki-Seki
Ki-Seki merged commit eede19b into MemTensor:dev Jul 10, 2025
16 checks passed
tianxing02 pushed a commit to tianxing02/MemOS that referenced this pull request Feb 24, 2026
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.

2 participants