The Wayback Machine - https://web.archive.org/web/20260424080450/https://github.com/angular/angular-cli/pull/19102
Skip to content

feat(@angular-devkit/build-angular): support following symlinked asset directories#19102

Merged
alan-agius4 merged 1 commit intoangular:masterfrom
clydin:assets-follow-symlinks
Oct 21, 2020
Merged

feat(@angular-devkit/build-angular): support following symlinked asset directories#19102
alan-agius4 merged 1 commit intoangular:masterfrom
clydin:assets-follow-symlinks

Conversation

@clydin
Copy link
Copy Markdown
Member

@clydin clydin commented Oct 16, 2020

By default subdirectories within a symlinked directory are not searched by a glob. The new followSymlinks option for the longhand form of the assets browser builder option now allows opting in to search such subdirectories.

partially addresses #19086

@clydin clydin added the target: major This PR is targeted for the next major release label Oct 16, 2020
@google-cla google-cla Bot added the cla: yes label Oct 16, 2020
@alan-agius4
Copy link
Copy Markdown
Collaborator

Mind updating the CLI schema as well?

@clydin clydin force-pushed the assets-follow-symlinks branch from 1d75841 to b988b60 Compare October 21, 2020 18:20
@clydin clydin marked this pull request as ready for review October 21, 2020 18:20
@clydin clydin requested a review from alan-agius4 October 21, 2020 18:21
@clydin clydin force-pushed the assets-follow-symlinks branch from b988b60 to dada6d8 Compare October 21, 2020 18:40
Comment thread packages/angular_devkit/build_angular/src/browser/schema.json
Copy link
Copy Markdown
Collaborator

@alan-agius4 alan-agius4 left a comment

Choose a reason for hiding this comment

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

One nit otherwise LGTM

@clydin clydin force-pushed the assets-follow-symlinks branch from dada6d8 to 974fd1c Compare October 21, 2020 18:47
@clydin clydin added the action: merge The PR is ready for merge by the caretaker label Oct 21, 2020
…t directories

By default subdirectories within a symlinked directory are not searched by a glob.  The new `followSymlinks` option for the longhand form of the `assets` browser builder option now allows opting in to search such subdirectories.
@clydin clydin force-pushed the assets-follow-symlinks branch from 974fd1c to 1f7cd2d Compare October 21, 2020 18:54
@alan-agius4 alan-agius4 merged commit cc723d8 into angular:master Oct 21, 2020
@clydin clydin deleted the assets-follow-symlinks branch October 21, 2020 20:04
@angular-automatic-lock-bot
Copy link
Copy Markdown

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 Nov 21, 2020
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 target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

HTTPS · web.archive.org
← Home