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

support symlinks #762

Closed
wants to merge 2 commits into from
Closed

support symlinks #762

wants to merge 2 commits into from

Conversation

samdenty
Copy link

@samdenty samdenty commented Aug 10, 2022

This PR adds supports for listing the files inside a symlinked extension.

Before, it would include files that were ignored. And include the realpath, vs the fake path

Copy link
Member

@joaomoreno joaomoreno left a comment

Choose a reason for hiding this comment

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

This just looks risky, given all the string manipulation of paths. Could we have some concrete tests for this?

@samdenty
Copy link
Author

This just looks risky, given all the string manipulation of paths. Could we have some concrete tests for this?

How would I test this?

@joaomoreno
Copy link
Member

joaomoreno commented Aug 29, 2022

This PR adds supports for listing the files inside a symlinked extension.

In the test suite, create a test which scaffolds a symlinked extension and then uses vsce to package it. Ideally, it's a test which fails today in main and passes in your branch.

@joaomoreno
Copy link
Member

Closing due to being stale.

@joaomoreno joaomoreno closed this Dec 13, 2023
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.

None yet

2 participants