Skip to content

Conversation

csb324
Copy link

@csb324 csb324 commented Mar 18, 2020

Fixes #228!

It looked like google-auth-library was about five major versions behind, and google had made some changes to how it all worked. So I...

  • updated google-auth-library,
  • then updated the relevant files to use the new google-auth-library interfaces
  • then updated the tests so that the tests will reveal this kind of problem in the future
  • ... which led to a teeny refactor of the gsheets.ts file (which was mostly done to make stubbing easier).

And now the package is working for me again! The one thing I couldn't solve is that the linter doesn't like my async arrow function test, so I imagine the CI tests here will fail. I couldn't figure out what that was about, considering the tests work, and the google-oauth test suite uses the same syntax without complaining.

oauthStub.restore();
});

it('should at least consider running', async() => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
it('should at least consider running', async() => {
it('should at least consider running', () => {

Copy link
Contributor

Choose a reason for hiding this comment

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

This seems to work on my end but I'm not totally sure about it..

@denar90
Copy link
Collaborator

denar90 commented Mar 24, 2020

Hey @csb324. Thank you for the PR 👍 Do you think we can have failing CI fixed? I'll be more than happy to mere your fix :)

@nathanmillar16
Copy link

Hi. I left a comment on the PR. Was you able to write to a google-sheet @csb324 ? Thanks

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Google sheets API error?

4 participants