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

Allow checkForModule to match child module names w/ arbitrary paths #208

Merged
merged 1 commit into from
Dec 15, 2023

Conversation

quinnj
Copy link
Contributor

@quinnj quinnj commented Dec 14, 2023

We have a use-case where we have a root gradle project (e.g. in root/), and then gradle subprojects that live in various subdirectories (e.g. root/apps/cool-app, root/libs/api/cool-lib).

The problem is that our root level build.gradle then has entries like:

include 'apps:cool-app'
include 'libs:api:cool-lib'

And the current checkForModule assumes the subproject always lives in the same directory as root.

With the proposed regex change here, we can ensure the nx executors work correctly for nx build cool-app.

@tinesoft
Copy link
Owner

Hi @quinnj

Thanks for using the plugin and for your PR.
It looks good to me!

Please, just change the commit message so it matches the conventional commit convention of the project, as sunch:

git commit --amend -m 'fix(common-jvm): fix `checkForModule` util to match gradle child module names w/ arbitrary paths'
git push origin head --force

This will allow semantic release.

Thanks

@quinnj
Copy link
Contributor Author

quinnj commented Dec 15, 2023

Thanks for the review @tinesoft! Pushed!

Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@tinesoft tinesoft merged commit 759c02f into tinesoft:develop Dec 15, 2023
5 of 6 checks passed
Copy link

🎉 This PR is included in version 1.1.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version 9.2.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version 5.2.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version 7.2.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version 3.2.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version 2.2.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version 2.2.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version 2.2.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version 2.2.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants