Skip to content

Fix/esm compatibility and workflow#10

Merged
utkarsh232005 merged 2 commits intoKDM-cli:mainfrom
utkarsh232005:fix/esm-compatibility-and-workflow
May 10, 2026
Merged

Fix/esm compatibility and workflow#10
utkarsh232005 merged 2 commits intoKDM-cli:mainfrom
utkarsh232005:fix/esm-compatibility-and-workflow

Conversation

@utkarsh232005
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings May 10, 2026 20:28
@utkarsh232005 utkarsh232005 merged commit 752ad1e into KDM-cli:main May 10, 2026
2 checks passed
Copy link
Copy Markdown

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.

Pull request overview

This PR updates the npm publish GitHub Actions workflow to adjust when dependencies are installed and to optionally commit package-lock.json changes during the release process.

Changes:

  • Moves the dependency installation step to occur after npm version.
  • Adds a step to commit package-lock.json if it changes after installation.

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

Comment on lines 37 to 38
- name: Bump version
run: "npm version ${{ github.event.inputs.release_type }} -m 'chore: release v%s'"
Comment on lines +41 to +47
run: npm install

- name: Commit lockfile if changed
run: |
git add package-lock.json || true
git diff --cached --quiet || git commit -m 'chore: update package-lock.json'


- name: Commit lockfile if changed
run: |
git add package-lock.json || true
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