Skip to content

refactor: remove js_image_layer bazel5 legacy symlink detection#1610

Merged
jbedard merged 1 commit into
aspect-build:2.xfrom
jbedard:2x-rm-symlink-detection
Apr 7, 2024
Merged

refactor: remove js_image_layer bazel5 legacy symlink detection#1610
jbedard merged 1 commit into
aspect-build:2.xfrom
jbedard:2x-rm-symlink-detection

Conversation

@jbedard

@jbedard jbedard commented Apr 6, 2024

Copy link
Copy Markdown
Member

Type of change

  • Refactor (a code change that neither fixes a bug or adds a new feature)

Test plan

  • Covered by existing test cases

@jbedard jbedard force-pushed the 2x-rm-symlink-detection branch from ee9a5cf to 6886a04 Compare April 6, 2024 22:42
@jbedard jbedard marked this pull request as ready for review April 7, 2024 00:29
@jbedard jbedard requested review from gregmagolan and thesayyn April 7, 2024 00:29

@thesayyn thesayyn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

you need to update rule impl as well

@jbedard jbedard force-pushed the 2x-rm-symlink-detection branch from 6886a04 to 48af645 Compare April 7, 2024 00:35
@jbedard

jbedard commented Apr 7, 2024

Copy link
Copy Markdown
Member Author

you need to update rule impl as well

Good point, thanks! (and updated...)

@jbedard jbedard force-pushed the 2x-rm-symlink-detection branch from 48af645 to 981c67e Compare April 7, 2024 01:27

@gregmagolan gregmagolan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🌮

@jbedard jbedard merged commit caac704 into aspect-build:2.x Apr 7, 2024
@gregmagolan gregmagolan mentioned this pull request Apr 29, 2024
21 tasks
@jbedard jbedard deleted the 2x-rm-symlink-detection branch June 24, 2025 04:52
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.

3 participants