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

feat: Add support for customizable projectFileExtensions #20397

Merged
merged 12 commits into from
Nov 11, 2024

Conversation

caalador
Copy link
Contributor

@caalador caalador commented Nov 4, 2024

Fixes #19527

@caalador caalador force-pushed the feat/19527-project-file-extensions branch from fc9afca to 27a0697 Compare November 4, 2024 12:42
Copy link

github-actions bot commented Nov 4, 2024

Test Results

1 142 files  ±0  1 142 suites  ±0   1h 23m 53s ⏱️ - 5m 19s
7 487 tests +2  7 437 ✅ +2  50 💤 ±0  0 ❌ ±0 
7 845 runs   - 9  7 785 ✅  - 9  60 💤 ±0  0 ❌ ±0 

Results for commit 00cb72b. ± Comparison against base commit 4a08a10.

♻️ This comment has been updated with latest results.

Copy link
Member

@tltv tltv left a comment

Choose a reason for hiding this comment

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

Should also gradle plugin's PrepareFrontendInputProperties.kt be updated for the new property?
Also GenerateNpmBOMMojo maven goal could need to use this new property.

@caalador caalador force-pushed the feat/19527-project-file-extensions branch from d4b4b8e to 7fb996a Compare November 6, 2024 06:10
Fix initParam name, update javadocs
@caalador caalador force-pushed the feat/19527-project-file-extensions branch from 7fb996a to af99ce3 Compare November 6, 2024 06:54
Comment on lines 125 to 126
System.out.println(
"=== '" + options.getFrontendExtraFileExtensions() + "'");
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
System.out.println(
"=== '" + options.getFrontendExtraFileExtensions() + "'");
System.out.println(
"=== '" + options.getFrontendExtraFileExtensions() + "'");

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I forgot to push the removal =P

@caalador
Copy link
Contributor Author

caalador commented Nov 7, 2024

Added input property for the gradle plugin.

@caalador caalador force-pushed the feat/19527-project-file-extensions branch from 37bb8b9 to 671380c Compare November 7, 2024 05:57
tltv
tltv previously approved these changes Nov 7, 2024
Copy link
Member

@tltv tltv left a comment

Choose a reason for hiding this comment

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

LGTM. After conflicts are resolved.

# Conflicts:
#	flow-plugins/flow-gradle-plugin/src/main/kotlin/com/vaadin/gradle/VaadinFlowPluginExtension.kt
#	flow-plugins/flow-maven-plugin/src/main/java/com/vaadin/flow/plugin/maven/FlowModeAbstractMojo.java
#	flow-plugins/flow-plugin-base/src/main/java/com/vaadin/flow/plugin/base/BuildFrontendUtil.java
#	flow-plugins/flow-plugin-base/src/main/java/com/vaadin/flow/plugin/base/PluginAdapterBase.java
#	flow-server/src/main/java/com/vaadin/flow/server/frontend/Options.java
#	flow-server/src/main/java/com/vaadin/flow/server/startup/AbstractConfigurationFactory.java
#	vaadin-dev-server/src/main/java/com/vaadin/base/devserver/startup/DevModeInitializer.java
Copy link

sonarcloud bot commented Nov 8, 2024

@mshabarov mshabarov merged commit 717d482 into main Nov 11, 2024
26 checks passed
@mshabarov mshabarov deleted the feat/19527-project-file-extensions branch November 11, 2024 12:41
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Vaadin 24.6.0.alpha3 and is also targeting the upcoming stable 24.6.0 version.

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.

Include .scss and .sass in stats.json hashes
6 participants