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

[Packager] Windows support for Packager - Blacklist changes #893

Closed
wants to merge 1 commit into from
Closed

[Packager] Windows support for Packager - Blacklist changes #893

wants to merge 1 commit into from

Conversation

joewood
Copy link

@joewood joewood commented Apr 17, 2015

Another Pull Request implementing the changes in issue #468 - Enabled Packager to run on Windows
This change relates to the blacklist fixes. It includes the path conversion for blacklist and changes to the default watched directory. It has no impact on Mac OSX.

@joewood joewood closed this Apr 17, 2015
@joewood joewood reopened this Apr 17, 2015
@joewood joewood closed this Apr 17, 2015
@joewood joewood reopened this Apr 17, 2015
@joewood joewood closed this Apr 17, 2015
@joewood joewood reopened this Apr 17, 2015
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 17, 2015
// Don't forget to everything listed here to `testConfig.json`
// modulePathIgnorePatterns.
var sharedBlacklist = [
__dirname,
'website',
'/.git', // added because nodeWatcher does not ignore hidden files
Copy link
Contributor

Choose a reason for hiding this comment

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

recently fixed this error in sane -- the file watcher.

Copy link
Contributor

Choose a reason for hiding this comment

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

but it's fine to keep

Copy link
Author

Choose a reason for hiding this comment

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

Great, also noticed you fixed the NodeWatcher creation to pass an object with glob as a field, which seem to fix an issue where the glob pattern was being ignored.

@amasad
Copy link
Contributor

amasad commented Apr 17, 2015

This looks good. Will pull shortly

@brentvatne
Copy link
Collaborator

@amasad - did this ever get pulled?

@brentvatne brentvatne changed the title Windows support for Packager - Blacklist changes [Packager] Windows support for Packager - Blacklist changes Jun 1, 2015
@amasad
Copy link
Contributor

amasad commented Jun 2, 2015

Looks like it wasn't :/ I'll pull it

@a2 a2 closed this in 19e3239 Jul 2, 2015
cpojer pushed a commit to facebook/metro that referenced this pull request Jan 26, 2017
Summary:
Another Pull Request implementing the changes in issue #468 - Enabled Packager to run on Windows
This change relates to the blacklist fixes. It includes the path conversion for blacklist and changes to the default watched directory.  It has no impact on Mac OSX.
Closes facebook/react-native#893
Github Author: Joe Wood <joewood>

Test Plan: Imported from GitHub, without a `Test Plan:` line.
ayushjainrksh pushed a commit to MLH-Fellowship/react-native that referenced this pull request Jul 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants