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

Improve TypeScript types (part 4) #402

Merged
merged 4 commits into from
Sep 19, 2018
Merged

Improve TypeScript types (part 4) #402

merged 4 commits into from
Sep 19, 2018

Conversation

JustinBeckwith
Copy link
Contributor

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 19, 2018
@ghost ghost assigned JustinBeckwith Sep 19, 2018
@@ -1644,7 +1653,8 @@ class Bucket extends ServiceObject {
},
(err, resp) => {
if (err) {
callback(err, null, null, resp);
// tslint:disable-next-line:no-any
(callback as any)(err, null, null, resp);

This comment was marked as spam.

This comment was marked as spam.

@JustinBeckwith
Copy link
Contributor Author

Side note - my system tests appear to be passing?

getFiles(
queryOrCallback?: GetFilesOptions|GetFilesCallback,
callback?: GetFilesCallback): void|Promise<[File[]]> {
let query = typeof queryOrCallback === 'object' ? queryOrCallback : {};

This comment was marked as spam.

This comment was marked as spam.

@ghost ghost assigned jkwlui Sep 19, 2018
@jkwlui
Copy link
Member

jkwlui commented Sep 19, 2018

For system-test, it's complaining that it couldn't write to /home/node/.config

I am guessing it's because we brought in .circleci/config.yml from synth that had user: node in the docker config that's causing some permission issues. I'm investigating via SSH.

Okay it's not that reason. But the system-test did start failing from commit 9f65f9d

@jkwlui
Copy link
Member

jkwlui commented Sep 19, 2018

Samples test has always been flaky on that one test where it complains about unlink downloaded.txt..

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.

3 participants