Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(build): update version bumper #1990

Closed
wants to merge 6 commits into from

Conversation

fshovchko
Copy link
Contributor

After looking up the solutions for automatic version update, it was decided to enhance our own already made solution
#1529

@fshovchko fshovchko requested a review from ala-n October 19, 2023 20:31
@ala-n
Copy link
Collaborator

ala-n commented Oct 23, 2023

It looks like we need to revisit the approach. As was discovered in #2003 the version bumping can be handled via existing npm processes, nevertheless we still need a solution for meta information update which bump.js can handle

bump.config.yml Outdated Show resolved Hide resolved
@fshovchko fshovchko requested a review from ala-n October 30, 2023 14:47
@@ -9,7 +9,7 @@
"test": "jest",
"clean": "rimraf dist",
"build": "npm run clean && tsc --project tsconfig.json",
"version": "node ../build/bump.js"
"version": "node ../build/bump.js $npm_package_version"
Copy link
Collaborator

@ala-n ala-n Nov 2, 2023

Choose a reason for hiding this comment

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

You don't need env $npm_package_version here, because

  1. it is already in process.env, while mentioning it here produces problems on windows machines (out of cross-env runner)
  2. action in version hook is executed after native npm version process so the actual version is also presented inside current package.json

- filePath: 'package.json'
patterns:
- '"name"\s*:\s*"\@exadel\/eslint-plugin-esl"[^{}]*"version"\s*:\s*"(\d+\.\d+\.\d+)"'
- '"@exadel\/esl\": \"\^(\d+\.\d+\.\d+)\"'
Copy link
Collaborator

@ala-n ala-n Nov 2, 2023

Choose a reason for hiding this comment

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

It appears to be incorrect, as I mentioned earlier.
The native npm version processes the version section and dependencies without our interventions.

@ala-n
Copy link
Collaborator

ala-n commented Nov 2, 2023

After a deeper investigation it appears to be that we no longer require any additional actions at all.
The eslint section in package.json is redundant.
Test 4.13.0 build of eslint plugin published. There are no additional issues with versioning.

@ala-n ala-n closed this Nov 2, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Nov 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants