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: update log message after versions.json separation #14083

Merged
merged 1 commit into from
Jun 28, 2022

Conversation

ZheSun88
Copy link
Contributor

after #14014 fix, one log message needs to be updated.

@ZheSun88 ZheSun88 requested a review from taefi June 28, 2022 05:42
@sonarqubecloud
Copy link

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 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@github-actions
Copy link

github-actions bot commented Jun 28, 2022

Unit Test Results

   982 files  ±0     982 suites  ±0   53m 39s ⏱️ + 1m 1s
6 322 tests ±0  6 274 ✔️ ±0  48 💤 ±0  0 ±0 
6 584 runs  ±0  6 530 ✔️ ±0  54 💤 ±0  0 ±0 

Results for commit cdb314c. ± Comparison against base commit 9747bfc.

♻️ This comment has been updated with latest results.

@taefi taefi merged commit 1b86801 into master Jun 28, 2022
@taefi taefi deleted the ZheSun88-patch-5 branch June 28, 2022 07:19
taefi pushed a commit that referenced this pull request Jun 28, 2022
taefi pushed a commit that referenced this pull request Jun 29, 2022
taefi added a commit that referenced this pull request Jun 29, 2022
* fix: separate core and commercial components version files (#14014)

* fix: separate core and commercial components version files

Fixes: #13749

Previously, the vaadin_versions.json contained
both core and commercial components listed
which led to including them all in package.json
even if the user only used vaadin-core. Now
platform generates vaadin-core-version.json in
vaadin-core and vaadin-version.json to include
commercial components in vaadin artifact.
This commit will adopt having separate files
while generating package.json and pinning the
platform dependencies.

(cherry picked from commit 23f5496)

* test: add tests for handling separate versions in NodeUpdater (#14079)

Tests are added for changes done in PR
#14014

(cherry picked from commit 7140d32)

* chore: update log message after versions.json separation (#14083)

(cherry picked from commit 1b86801)

Co-authored-by: Zhe Sun <[email protected]>
vaadin-bot pushed a commit that referenced this pull request Jun 29, 2022
* fix: separate core and commercial components version files (#14014)

* fix: separate core and commercial components version files

Fixes: #13749

Previously, the vaadin_versions.json contained
both core and commercial components listed
which led to including them all in package.json
even if the user only used vaadin-core. Now
platform generates vaadin-core-version.json in
vaadin-core and vaadin-version.json to include
commercial components in vaadin artifact.
This commit will adopt having separate files
while generating package.json and pinning the
platform dependencies.

(cherry picked from commit 23f5496)

* test: add tests for handling separate versions in NodeUpdater (#14079)

Tests are added for changes done in PR
#14014

(cherry picked from commit 7140d32)

* chore: update log message after versions.json separation (#14083)

(cherry picked from commit 1b86801)

Co-authored-by: Zhe Sun <[email protected]>
taefi added a commit that referenced this pull request Jul 1, 2022
…14104)

* fix: separate core and commercial components version files (#14014)

* fix: separate core and commercial components version files

Fixes: #13749

Previously, the vaadin_versions.json contained
both core and commercial components listed
which led to including them all in package.json
even if the user only used vaadin-core. Now
platform generates vaadin-core-version.json in
vaadin-core and vaadin-version.json to include
commercial components in vaadin artifact.
This commit will adopt having separate files
while generating package.json and pinning the
platform dependencies.

(cherry picked from commit 23f5496)

* test: add tests for handling separate versions in NodeUpdater (#14079)

Tests are added for changes done in PR
#14014

(cherry picked from commit 7140d32)

* chore: update log message after versions.json separation (#14083)

(cherry picked from commit 1b86801)

Co-authored-by: Zhe Sun <[email protected]>

Co-authored-by: Soroosh Taefi <[email protected]>
taefi added a commit that referenced this pull request Jul 1, 2022
…(CP: 9.0)

This is a manual cherry-pick to collect
all changes that are done in #14014,
#14079, and #14083.

Related to #14092
taefi added a commit that referenced this pull request Jul 1, 2022
…(CP: 2.7)

This is a manual cherry-pick to collect
all changes that are done in #14014,
#14079, and #14083.

Related to #14092
taefi added a commit that referenced this pull request Jul 1, 2022
…(CP: 9.0) (#14123)

* fix: separate core and commercial components version files (#14014) (CP: 9.0)

This is a manual cherry-pick to collect
all changes that are done in #14014,
#14079, and #14083.

Related to #14092
mcollovati pushed a commit that referenced this pull request Jul 1, 2022
…(CP: 2.7) (#14124)

This is a manual cherry-pick to collect
all changes that are done in #14014,
#14079, and #14083.

Related to #14092
vaadin-bot pushed a commit that referenced this pull request Jul 1, 2022
…(CP: 2.7) (#14124)

This is a manual cherry-pick to collect
all changes that are done in #14014,
#14079, and #14083.

Related to #14092
mcollovati pushed a commit that referenced this pull request Jul 1, 2022
…(CP: 2.7) (#14124) (#14126)

This is a manual cherry-pick to collect
all changes that are done in #14014,
#14079, and #14083.

Related to #14092
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants