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

Support for default source and public paths #85

Merged
merged 1 commit into from
Dec 9, 2024

Conversation

vmj
Copy link
Contributor

@vmj vmj commented Dec 9, 2024

Hi,

we've been using the v1 version of this plugin for about a 6 months I guess (thanks!), and we are considering moving to v2.

We have multiple GWT modules in one Gradle source set, and only one of them has an entry-point element. So guessing the source path from that doesn't work for us. We could add the <source path="client"/> to all the modules, but that seems unnecessary, because:

As mentioned in
https://www.gwtproject.org/doc/latest/DevGuideOrganizingProjects.html if no source or public element is defined in a module XML file, the client and public subpackage is implicitly added to the source/public path as if <source path="client" /> or <public path="public"> had been found in the XML.

So, that is what this PR is about. What do you think?

As mentioned in
https://www.gwtproject.org/doc/latest/DevGuideOrganizingProjects.html
if no `source` or `public` element is defined in a module XML file,
the `client` and `public` subpackage is implicitly added to the
source/public path as if `<source path="client" />` or `<public
path="public">` had been found in the XML.
Copy link
Owner

@jiakuan jiakuan left a comment

Choose a reason for hiding this comment

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

Looks great!
Thanks

@jiakuan jiakuan merged commit efb5572 into jiakuan:main Dec 9, 2024
1 check failed
@jiakuan
Copy link
Owner

jiakuan commented Dec 9, 2024

From the failing tests, I can see we need to check the existence of the folders before adding them to the list.
I've pushed a minor fix.
And, have released a new version:

plugins {
  id("org.docstr.gwt") version "2.1.4"
}

@vmj vmj deleted the default-paths branch December 9, 2024 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants