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

feat: use small HTTP dependency #416

Merged
merged 1 commit into from
Oct 12, 2018
Merged

feat: use small HTTP dependency #416

merged 1 commit into from
Oct 12, 2018

Conversation

fhinkel
Copy link
Contributor

@fhinkel fhinkel commented Sep 28, 2018

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 28, 2018
@ghost ghost assigned fhinkel Sep 28, 2018
@JustinBeckwith
Copy link
Contributor

Nice! The system and sample tests are currently down for this one, and we would need to get those passing first though :)

Copy link
Contributor

@JustinBeckwith JustinBeckwith left a comment

Choose a reason for hiding this comment

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

Assuming all CI is passing

package.json Outdated
@@ -94,6 +94,7 @@
"request": "^2.88.0",

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@fhinkel
Copy link
Contributor Author

fhinkel commented Sep 28, 2018

I was wondering how that tests ran so fast. No problem with waiting. Who is working on fixing the system tests?

@fhinkel fhinkel force-pushed the withTeeny branch 2 times, most recently from 676d053 to 46883d5 Compare September 28, 2018 22:52
@ghost ghost assigned JustinBeckwith Oct 1, 2018
@jkwlui jkwlui added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 4, 2018
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 4, 2018
@JustinBeckwith
Copy link
Contributor

@fhinkel I think we have the tests worked out over here. Can you resolve conflicts and give this another go? Thanks!

@fhinkel
Copy link
Contributor Author

fhinkel commented Oct 10, 2018

@kinwa91 Can I go ahead and merge or are we in the middle of releasing?

@fhinkel fhinkel merged commit c2c1382 into master Oct 12, 2018
fhinkel added a commit that referenced this pull request Oct 12, 2018
System-tests broke with that commit.

This reverts commit c2c1382.
@fhinkel fhinkel deleted the withTeeny branch October 12, 2018 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants