Skip to content

chore: use --frozen-lockfile option for pnpm install in workflows#263

Merged
DonOmalVindula merged 1 commit intoasgardeo:mainfrom
DonOmalVindula:npm-nov-demote
Nov 27, 2025
Merged

chore: use --frozen-lockfile option for pnpm install in workflows#263
DonOmalVindula merged 1 commit intoasgardeo:mainfrom
DonOmalVindula:npm-nov-demote

Conversation

@DonOmalVindula
Copy link
Copy Markdown
Contributor

Purpose

This pull request updates several GitHub workflow files to improve dependency installation reliability and ensure consistency in YAML formatting. The main changes include switching to pnpm install --frozen-lockfile for dependency installation and standardizing string quoting throughout workflow configuration files.

Dependency installation reliability:

  • Updated all workflow steps that install dependencies to use pnpm install --frozen-lockfile instead of pnpm install, ensuring lockfile integrity and preventing accidental dependency changes. This affects .github/workflows/npm-audit.yml, .github/workflows/pr-builder.yml, and .github/workflows/release.yml. [1] [2] [3] [4] [5] [6]

YAML formatting and consistency:

  • Standardized the use of single quotes for string values in workflow files, including branch names, paths to ignore, and commit messages, to improve readability and maintain consistency. [1] [2] [3] [4]

Related Issues

  • N/A

Related PRs

  • N/A

Checklist

  • Followed the CONTRIBUTING guidelines.
  • Manual test round performed and verified.
  • Documentation provided. (Add links if there are any)
  • Unit tests provided. (Add links if there are any)

Security checks

@asgardeo-github-bot
Copy link
Copy Markdown

⚠️ No Changeset found

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go.

If these changes should result in a version bump, you need to add a changeset.

Refer Release Documentation to learn how to add a changeset.

@DonOmalVindula DonOmalVindula merged commit 290893d into asgardeo:main Nov 27, 2025
4 of 6 checks passed
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.

3 participants