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

feat(bazel): prefix private-export (barred-latin-o) symbols #23007

Closed

Conversation

alexeagle
Copy link
Contributor

This allows a bundle index to be re-exported by a higher-level module without fear of collisions.
Under bazel, we always set the prefix to be underscore-joined workspace, package, label

@@ -99,6 +99,10 @@ export interface CompilerOptions extends ts.CompilerOptions {
// meaningful when `flatModuleOutFile` is also supplied. It is otherwise ignored.
flatModuleId?: string;

// A prefix to insert in generated private symbols, e.g. for "my_prefix_" we
// would generate private symbols named like `ɵmy_prefix_a`.
privateSymbolPrefix?: string;
Copy link
Contributor

Choose a reason for hiding this comment

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

Rename to flatModulePrivateSymbolPrefix

@alexeagle alexeagle added the target: major This PR is targeted for the next major release label Mar 26, 2018
Copy link
Member

@jelbourn jelbourn left a comment

Choose a reason for hiding this comment

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

LGTM

@mary-poppins
Copy link

You can preview df6e5ec at https://pr23007-df6e5ec.ngbuilds.io/.

@alexeagle alexeagle force-pushed the ng_package_namespace_privates branch from d1e4fd7 to cdb2674 Compare March 26, 2018 21:46
@mary-poppins
Copy link

You can preview d1e4fd7 at https://pr23007-d1e4fd7.ngbuilds.io/.

@mary-poppins
Copy link

You can preview cdb2674 at https://pr23007-cdb2674.ngbuilds.io/.

@alexeagle alexeagle force-pushed the ng_package_namespace_privates branch from cdb2674 to 0c96f97 Compare March 26, 2018 22:59
@mary-poppins
Copy link

You can preview 0c96f97 at https://pr23007-0c96f97.ngbuilds.io/.

This allows a bundle index to be re-exported by a higher-level module without fear of collisions.
Under bazel, we always set the prefix to be underscore-joined workspace, package, label
@alexeagle alexeagle force-pushed the ng_package_namespace_privates branch from 0c96f97 to c395536 Compare March 26, 2018 23:28
@mary-poppins
Copy link

You can preview c395536 at https://pr23007-c395536.ngbuilds.io/.

@alexeagle alexeagle added the action: merge The PR is ready for merge by the caretaker label Mar 27, 2018
@matsko matsko closed this in 27e14b2 Mar 27, 2018
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker cla: yes target: major This PR is targeted for the next major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants