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

Revert pull request #1714 #1872

Merged
merged 2 commits into from Aug 22, 2018
Merged

Revert pull request #1714 #1872

merged 2 commits into from Aug 22, 2018

Conversation

@grokys
Copy link
Contributor

@grokys grokys commented Aug 22, 2018

After testing #1714 was shown to cause problems with the extension. Although we think these problems should be fixed by #1871, we've decided that we need more time for testing.

We're reverting #1714 for now, but this PR will make it into our next release after we can be certain that it's not caused other issues.

Pinging @maikebing for visibility - thanks! Your work is really appreciated and we'll make sure to get it into the next release, we just need a little more time :)

This reverts commit 3b501b0, reversing
changes made to d3641f4.
@shana
shana approved these changes Aug 22, 2018
Copy link
Contributor

@meaghanlewis meaghanlewis left a comment

LGTM

@meaghanlewis meaghanlewis merged commit 5e27021 into master Aug 22, 2018
2 checks passed
2 checks passed
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
@meaghanlewis meaghanlewis deleted the revert/1714 branch Aug 22, 2018
StanleyGoldman added a commit that referenced this pull request Aug 30, 2018
This reverts commit 5e27021, reversing
changes made to 23e48d1.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.