Skip to content

Update configuration.adoc #4267

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

Merged
merged 6 commits into from
Aug 8, 2025
Merged

Update configuration.adoc #4267

merged 6 commits into from
Aug 8, 2025

Conversation

OlliTietavainenVaadin
Copy link
Member

Clarified the language of the Core & Pro Vaadin React Components chapter

OlliTietavainenVaadin and others added 2 commits April 24, 2025 12:00
Clarified the language of the Core & Pro Vaadin React Components chapter
Copy link

github-actions bot commented Apr 28, 2025

AI Language Review

The modified file, "Configuration," has made a few improvements for clarity and consistency. However, one issue requires improvement:

  1. In the section "Enable React 19," the phrase "using feature flag" should be revised for grammatical correctness and clarity. It could be rephrased as "using a feature flag" to specify that a feature flag is being used.

@peholmst peholmst merged commit 3ad6c2a into v24 Aug 8, 2025
4 checks passed
@peholmst peholmst deleted the OlliTietavainenVaadin-patch-1 branch August 8, 2025 05:44
vaadin-bot pushed a commit that referenced this pull request Aug 8, 2025
* Update configuration.adoc

Clarified the language of the Core & Pro Vaadin React Components chapter

* AI language review fixes

---------

Co-authored-by: Manuel Carrasco Moñino <[email protected]>
Co-authored-by: Petter Holmström <[email protected]>
peholmst added a commit that referenced this pull request Aug 8, 2025
* Update configuration.adoc

Clarified the language of the Core & Pro Vaadin React Components chapter

* AI language review fixes

---------

Co-authored-by: Olli Tietäväinen <[email protected]>
Co-authored-by: Manuel Carrasco Moñino <[email protected]>
Co-authored-by: Petter Holmström <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-picked-main target/main cherry pick to main branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants