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

Question: Interested in a refactored version? #10

Open
karatekaneen opened this issue May 21, 2021 · 3 comments
Open

Question: Interested in a refactored version? #10

karatekaneen opened this issue May 21, 2021 · 3 comments

Comments

@karatekaneen
Copy link

Thank you for this great repo!

I made a fork a while back and to be able to add the functionality we needed I did a refactor to Typescript.

Major stuff i did:

  • Split up into multiple files for better readability
  • Converted to Typescript
  • Added support for multiple users and buckets.

I realize that some of the changes might be outside of the scope of this project but if you want I can open a PR?

@kolban-google
Copy link
Owner

Oooh ... let me stew on this a while. I don't speak "typescript" ... I'm just simple JavaScript shaped. Let me read on what the value (generically) is in Typescript and then contemplate whether I want to accept a complete migration to that variant. At a minimum, I'd be more than willing to add a line or two in the README pointing to your repo for a Typescript flavor ... but I'd also add a massive disclaimer that anyone using your code do so as a relationship between you and them and that would have NOTHING to do with Google.

@karatekaneen
Copy link
Author

I fully understand.
The value of using Typescript, for me at least, is that more errors are caught at the time of writing so I don't have to spend time looking for them. The great support for autocomplete in the editor doesn't hurt either.
The split in to multiple files can help future contributions from others since it's (in my opinion) easier to get an overview when each file has a more specified functionality as well as it will be easier to add unit tests.

I also realized that I forgot to mention one of the major changes I did, which was to remove the CLI-part, since we're not going to use it that way but I''m happy to add it back in in the submitted version if requested.

Regarding the disclaimer I think that's a great idea and I'll add that myself to my repo to minimize the risks of any misconceptions about the project's affiliation with Google.

@noahjustinsure
Copy link

personally I was able to get this fork up in running in no time and understood what was going on a bit better.

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

No branches or pull requests

3 participants