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

fix: add nodeVersion in gradle plugin settings #20059

Merged

Conversation

mrk-andreev
Copy link
Contributor

Description

Prior to this fix, the settings declared in Gradle were not being passed to the plugin. For example, changing the nodeVersion in build.gradle did not update the actual nodeVersion used.

Fixes # (issue)

Type of change

  • Bugfix

Checklist

  • I have read the contribution guide: https://vaadin.com/docs/latest/guide/contributing/overview/
  • I have added a description following the guideline.
  • The issue is created in the corresponding repository and I have referenced it.
  • I have added tests to ensure my change is effective and works as intended.
  • New and existing tests are passing locally with my change.
  • I have performed self-review and corrected misspellings.

There are no existed tests for this part of code. You can review code execution flow for review this changes.

Prior to this fix, the settings declared in Gradle were not being passed to the plugin. For example, changing the nodeVersion in build.gradle did not update the actual nodeVersion used.
@CLAassistant
Copy link

CLAassistant commented Sep 25, 2024

CLA assistant check
All committers have signed the CLA.

@mshabarov mshabarov added the Contribution PRs coming from the community or external to the team label Sep 27, 2024
Copy link
Contributor

@mshabarov mshabarov left a comment

Choose a reason for hiding this comment

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

Excellent catch, @mrk-andreev ! Thanks for contribution.

Copy link

Test Results

1 137 files  ± 0  1 137 suites  ±0   1h 11m 40s ⏱️ + 3m 49s
7 399 tests ± 0  7 349 ✅ + 1  50 💤 ±0  0 ❌ ±0 
7 771 runs  +17  7 711 ✅ +18  60 💤 ±0  0 ❌ ±0 

Results for commit eb9cd11. ± Comparison against base commit 69e2800.

Copy link

sonarcloud bot commented Sep 27, 2024

@mshabarov mshabarov merged commit b574c27 into vaadin:main Sep 27, 2024
26 of 28 checks passed
vaadin-bot pushed a commit that referenced this pull request Sep 27, 2024
Prior to this fix, the settings declared in Gradle were not being passed to the plugin. For example, changing the nodeVersion in build.gradle did not update the actual nodeVersion used.
vaadin-bot pushed a commit that referenced this pull request Sep 27, 2024
Prior to this fix, the settings declared in Gradle were not being passed to the plugin. For example, changing the nodeVersion in build.gradle did not update the actual nodeVersion used.
vaadin-bot pushed a commit that referenced this pull request Sep 27, 2024
Prior to this fix, the settings declared in Gradle were not being passed to the plugin. For example, changing the nodeVersion in build.gradle did not update the actual nodeVersion used.
vaadin-bot added a commit that referenced this pull request Sep 27, 2024
Prior to this fix, the settings declared in Gradle were not being passed to the plugin. For example, changing the nodeVersion in build.gradle did not update the actual nodeVersion used.

Co-authored-by: Mark Andreev <mark.andreev@gmail.com>
vaadin-bot added a commit that referenced this pull request Sep 27, 2024
Prior to this fix, the settings declared in Gradle were not being passed to the plugin. For example, changing the nodeVersion in build.gradle did not update the actual nodeVersion used.

Co-authored-by: Mark Andreev <mark.andreev@gmail.com>
vaadin-bot added a commit that referenced this pull request Sep 30, 2024
Prior to this fix, the settings declared in Gradle were not being passed to the plugin. For example, changing the nodeVersion in build.gradle did not update the actual nodeVersion used.

Co-authored-by: Mark Andreev <mark.andreev@gmail.com>
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.

NodeVersion params in gradle plugin settings don't effect to build
4 participants