Proposed v2.0.2 rewrite using [email protected]

[johnbeech]

I've raised an issue here: https://github.com/bassarisse/google-spreadsheet-to-json/issues/32 to advertise an updated version of the tool as a potential candidate to merge in the future.

Proposed change log:

2.0.2

Changes

• Add guard for parsing worksheets that don't contain cells; i.e. charts

2.0.1

Changes

• Fix bug with allWorksheets: true and added additional test

2.0.0

New

• Tests and CI using Github Actions to repo

Changes

• Fixed security vulnerabilities reported by npm audit
• Updated google-spreadsheet depenedncy
• Removed support for public published sheets due to deprecated authentication options
• Examples now use async await
• Changed internal workings to use 0 based indexes for rows and columns
• Changed internal workings to use cleaner async await code
• Introduced standard js as the enforced linter as part of the CI actions

[spatical]

@bassarisse I tested this updated code from @johnbeech and it seems to work great. In fact for my basic usage I was able to simply load his updated version without any code changes to my actual implementation code. I'd definitely recommend this update since the Google Spreadsheet module that is referenced has now passed the date that it was said to have been deprecated.

[gamedevsam]

Any reason why this shouldn't be merged? This library is kinda useless without these changes.