Skip to content

Conversation

bruce-one
Copy link
Contributor

What's the problem this PR addresses?

I'm in the process of upgrading an application (nexe) leveraging @yarnpkg/fslib from 2.x to 3.x.

In version 2 we were able to import CustomDir to create a custom implementation of opendir that was usable with a FakeFS, however this is no longer exported so doing so is no longer possible to leverage the CustomDir class.

...

How did you fix it?

This PR exports CustomDir in addition to opendir from opendir.ts.

...

Checklist

  • I have set the packages that need to be released for my changes to be effective.
  • I will check that all automated PR checks pass before the PR gets reviewed.

@bruce-one bruce-one force-pushed the export-custom-dir branch from 7519a8a to 88f1de7 Compare April 6, 2023 09:59
@merceyz merceyz self-requested a review April 6, 2023 10:10
@bruce-one bruce-one force-pushed the export-custom-dir branch from 88f1de7 to d2247de Compare April 6, 2023 20:59
To allow compatible implementations of `opendir` to be created by
consumers of this library.
@bruce-one bruce-one force-pushed the export-custom-dir branch from d2247de to e0fea10 Compare April 6, 2023 21:09
bruce-one added a commit to bruce-one/nexe that referenced this pull request Apr 7, 2023
@merceyz merceyz changed the title Export custom dir fix(fslib): export CustomDir Apr 7, 2023
@bruce-one bruce-one requested a review from RDIL April 7, 2023 22:43
bruce-one added a commit to bruce-one/nexe that referenced this pull request Apr 8, 2023
bruce-one added a commit to bruce-one/nexe that referenced this pull request Apr 8, 2023
bruce-one added a commit to bruce-one/nexe that referenced this pull request Apr 8, 2023
bruce-one added a commit to bruce-one/nexe that referenced this pull request Apr 8, 2023
@bruce-one
Copy link
Contributor Author

Thanks for the reviews @RDIL and @merceyz :-)

Do I need to do anything else to get this merged? No worries if not, just confirming :-)
Thanks :-)

@RDIL
Copy link
Member

RDIL commented Apr 9, 2023

For whatever reason the deploy preview for the website is failing so we can't merge it due to the permission setup.

@RDIL RDIL enabled auto-merge (squash) April 9, 2023 02:40
@arcanis arcanis disabled auto-merge April 24, 2023 15:06
@arcanis arcanis merged commit c70ff90 into yarnpkg:master Apr 24, 2023
merceyz pushed a commit that referenced this pull request Apr 24, 2023
* Whitespace.

* Export CustomDir class.

To allow compatible implementations of `opendir` to be created by
consumers of this library.
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.

4 participants