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

Avoid process spawn overhead for refreshing git status #15750

Merged
merged 11 commits into from Sep 27, 2017

Conversation

Projects
None yet
2 participants
@maxbrunsfeld
Contributor

maxbrunsfeld commented Sep 27, 2017

Fixes #9544
Depends on atom/git-utils#80

@maxbrunsfeld maxbrunsfeld changed the title from Convert GitRepository to JS to Avoid process spawn overhead for refreshing git status Sep 27, 2017

@maxbrunsfeld maxbrunsfeld merged commit a24f429 into master Sep 27, 2017

1 check was pending

ci/circleci CircleCI is running your tests
Details

@maxbrunsfeld maxbrunsfeld deleted the mb-remove-status-handler-task branch Sep 27, 2017

nathansobo added a commit to atom/fuzzy-finder that referenced this pull request Oct 10, 2017

Wait for status refresh in test, correct some assertions
This is based on 60b1f0c, which I am
back-porting in order hot-fix atom/atom#15750 to
Atom 1.21. The tests were converted to JS so I'm applying the same
changes from that commit manually rather than cherry-picking.

nathansobo added a commit to atom/fuzzy-finder that referenced this pull request Oct 10, 2017

Wait for status refresh in test, correct some assertions
This is based on 60b1f0c, which I am
back-porting in order hot-fix atom/atom#15750 to
Atom 1.21. The tests were converted to JS so I'm applying the same
changes from that commit manually rather than cherry-picking.

nathansobo added a commit that referenced this pull request Oct 10, 2017

Merge pull request #15750 from atom/mb-remove-status-handler-task
Avoid process spawn overhead for refreshing git status
@nathansobo

This comment has been minimized.

Show comment
Hide comment
@nathansobo

nathansobo Oct 10, 2017

Contributor

Hot-fixing this to 1.21.1.

Contributor

nathansobo commented Oct 10, 2017

Hot-fixing this to 1.21.1.

@maxbrunsfeld maxbrunsfeld referenced this pull request Nov 27, 2017

Closed

Uncaught (in promise) Error: Git head failed #15920

1 of 1 task complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment