Skip to content

[RUN-4642] Add new flag to mx create-module-package docs (10.23.0) #9455

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

Open
wants to merge 2 commits into
base: development
Choose a base branch
from

Conversation

atennapel
Copy link
Contributor

See #9387 for the corresponding Mx11 PR.

This is planned for 10.23.0.

@atennapel atennapel force-pushed the run/4642-mx-create-module-package-dep-sync-10 branch from c65e3f2 to 8abe538 Compare April 28, 2025 08:40
@quinntracy quinntracy self-assigned this Apr 28, 2025
@@ -63,6 +63,7 @@ These are the `OPTIONS`:
| `-l, --filter-required-libs` | Includes all the files except the userlibs that do not have an accompanying `[ModuleName].RequiredLib` file. |
| `-e, --exclude-files` | Excludes all the files that match the given regular expression. |
| `-d, --package-dir` | Exports the module package to the directory. |
| `-m, --exclude-managed-dependencies` | Excludes managed dependencies from synchronization and the exported module package. Note that this will prevent the module from being used if Gradle synchronization is disabled in Studio Pro. Available in 10.23 and later. In versions before 10.23 managed dependencies are not synchronized when using this command. |
Copy link
Member

Choose a reason for hiding this comment

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

In versions before 10.23 managed dependencies are not synchronized when using this command.
In versions before 10.23 they are never synchronized, so I think this should be worded differently.
Maybe just: 'In versions before 10.23, managed dependencies are never synchronized.'

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated.

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