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

Add support for build_extensions configuration of builders producing multiple files #647

Merged
merged 8 commits into from
Feb 23, 2023

Conversation

d-markey
Copy link
Contributor

@d-markey d-markey commented Feb 14, 2023

The validatedBuildExtensionsFrom() in utils.js currently rejects build_extensions configurations that use a list of strings rather than a single string.

This PR adds support for lists of strings.

d-markey added a commit to d-markey/squadron_builder that referenced this pull request Feb 19, 2023
- Breaking changes: several renamings, in particular the builder name which is now `squadron_builder:worker_builder`.
- Take builder options into account. In previous versions, builder options were ignored. Note that `source_gen` currently rejects builder configurations with multiple targets. Pull request dart-lang/source_gen#647 has been submitted to `source_gen` to support builders that produce multiple files.
- Added explicit option `with_finalizers` to force or disable code generation for finalization, and make finalization actually work.
- Split `build.yaml` in two to avoid interfering with the build process of client packages.
- Reorganized the source code to make it more readable and maintainable.
source_gen/CHANGELOG.md Outdated Show resolved Hide resolved
source_gen/lib/src/utils.dart Outdated Show resolved Hide resolved
source_gen/pubspec.yaml Outdated Show resolved Hide resolved
source_gen/lib/src/utils.dart Outdated Show resolved Hide resolved
@natebosch natebosch merged commit c4dfd1d into dart-lang:master Feb 23, 2023
@d-markey
Copy link
Contributor Author

Hello, thanks for the merge, I'd like to know also if you have any timeline in mind for releasing an update on pub.dev?

mosuem pushed a commit to dart-lang/build that referenced this pull request Dec 10, 2024
…g multiple files (dart-lang/source_gen#647)

Allow lists or single strings in the options.
mosuem pushed a commit to dart-lang/build that referenced this pull request Dec 10, 2024
…g multiple files (dart-lang/source_gen#647)

Allow lists or single strings in the options.
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