Skip to content

BLD: require pnpm, remove support for npm#72

Merged
shilorigins merged 4 commits intomainfrom
devagr/adopt-pnpm
Feb 27, 2026
Merged

BLD: require pnpm, remove support for npm#72
shilorigins merged 4 commits intomainfrom
devagr/adopt-pnpm

Conversation

@shilorigins
Copy link
Copy Markdown
Contributor

Description

  • add packageManager with pnpm version to package.json
  • remove package-lock.json
  • change npm to pnpm throughout README
  • use pnpm in linting workflow

Motivation

Following eed-web-application/elog-plus-frontend#187, this PR limits support to one package manager and pnpm is considered to be slightly faster and have a slightly smaller footprint.

Pre-merge checklist

  • Code works interactively
  • Code follows the style guide
  • Code contains descriptive docstrings
  • New/changed functions and methods are covered in the test suite where possible
  • Test suite passes locally
  • Test suite passes on GitHub Actions

@shilorigins shilorigins self-assigned this Feb 27, 2026
@shilorigins shilorigins requested a review from a team February 27, 2026 20:21
@shilorigins shilorigins merged commit 367f44f into main Feb 27, 2026
1 check passed
@shilorigins shilorigins deleted the devagr/adopt-pnpm branch February 27, 2026 20:47
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