Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Comments

Output Vite build directly into Maven target directory#30490

Merged
ahus1 merged 1 commit intokeycloak:mainfrom
jonkoops:dist-to-target
Jun 19, 2024
Merged

Output Vite build directly into Maven target directory#30490
ahus1 merged 1 commit intokeycloak:mainfrom
jonkoops:dist-to-target

Conversation

@jonkoops
Copy link
Contributor

Follow up of #30464, see #30464 (comment). Changes the output directory for Vite to the target directory of Maven, mitigating the need to copy these resources in Maven, and simplifying the build cache plugin configuration.

@ahus1
Copy link
Member

ahus1 commented Jun 19, 2024

When doing this, removing the dist entry from .gitignore might be possible as well.

Disadvantage is that it might be annoying while we switch between branches, and the advantage that we'll know if something re-appears in that folder.

@jonkoops
Copy link
Contributor Author

When doing this, removing the dist entry from .gitignore might be possible as well.

Looks like some of the libraries still use dist as a directory, so we'll have to keep it around.

@ahus1 ahus1 merged commit 2b35b44 into keycloak:main Jun 19, 2024
@ahus1
Copy link
Member

ahus1 commented Jun 19, 2024

Thank you for this simplification - this would also avoid copying around files which might even speed up the build a little.

@jonkoops jonkoops deleted the dist-to-target branch June 25, 2024 11:54
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.

3 participants