Skip to content

fix: Write version.json from the currently constructed package.json (#14696) (CP: 23.2)#14709

Merged
mshabarov merged 1 commit into
23.2from
cherry-pick-14696-to-23.2-1664529256308
Sep 30, 2022
Merged

fix: Write version.json from the currently constructed package.json (#14696) (CP: 23.2)#14709
mshabarov merged 1 commit into
23.2from
cherry-pick-14696-to-23.2-1664529256308

Conversation

@vaadin-bot

Copy link
Copy Markdown
Collaborator

No description provided.

…14696)

The earlier logic was like
1. Read package.json
2. Modify package.json in memory
3. Generate version.json based on package.json file
4. Write modified package.json to file

Where step 3 ignored what was done in step 2
@sonarqubecloud

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

@github-actions

Copy link
Copy Markdown

Unit Test Results

   929 files  ±0     929 suites  ±0   50m 54s ⏱️ - 2m 38s
6 054 tests ±0  6 000 ✔️ ±0  54 💤 ±0  0 ±0 
6 282 runs  ±0  6 221 ✔️ ±0  61 💤 ±0  0 ±0 

Results for commit 3562434. ± Comparison against base commit 4877e5f.

@mshabarov mshabarov merged commit 847f2a3 into 23.2 Sep 30, 2022
@mshabarov mshabarov deleted the cherry-pick-14696-to-23.2-1664529256308 branch September 30, 2022 13:04
@vaadin-bot

Copy link
Copy Markdown
Collaborator Author

This ticket/PR has been released with Vaadin 23.2.3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants