Skip to content

Align package manager artifacts with npm docs and confirm checklist path clarity#31

Merged
maehr merged 2 commits intomainfrom
copilot/fix-issue-27-29
Mar 6, 2026
Merged

Align package manager artifacts with npm docs and confirm checklist path clarity#31
maehr merged 2 commits intomainfrom
copilot/fix-issue-27-29

Conversation

Copy link
Contributor

Copilot AI commented Mar 6, 2026

Pull request

Proposed changes

The template currently documents npm-based setup while still carrying a pnpm-lock.yaml, which creates mixed package-manager signals. This change removes the pnpm lockfile and confirms the checklist already uses the explicit .github/ISSUE_TEMPLATE/config.yml path.

  • Issue coverage

    • Removed package manager inconsistency by deleting pnpm-lock.yaml.
    • Verified no README change needed for checklist file path: it already points to .github/ISSUE_TEMPLATE/config.yml.
  • Repository delta

    • Deleted: pnpm-lock.yaml
  • Example (artifact alignment)

    - pnpm-lock.yaml

Types of changes

  • New feature (non-breaking change which adds functionality).
  • Enhancement (non-breaking change which enhances functionality)
  • Bug Fix (non-breaking change which fixes an issue).
  • Breaking change (fix or feature that would cause existing functionality to change).

Checklist

  • I have read the README document.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: maehr <14755525+maehr@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix issues 27 and 29 Align package manager artifacts with npm docs and confirm checklist path clarity Mar 6, 2026
Copilot finished work on behalf of maehr March 6, 2026 07:07
@maehr maehr marked this pull request as ready for review March 6, 2026 07:08
Copilot AI review requested due to automatic review settings March 6, 2026 07:08
@maehr maehr merged commit 4d958d0 into main Mar 6, 2026
4 checks passed
@maehr maehr deleted the copilot/fix-issue-27-29 branch March 6, 2026 07:09
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

Documentation and lock file inconsistency: pnpm vs npm in install instructions

3 participants