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

Docs update and other minor fixes #3

Merged
merged 3 commits into from
Jan 23, 2023

Conversation

mikemaccana
Copy link
Contributor

Thanks for making this! Was speaking to Jacob and he mentioned this URL Aggregator is the best source of tokens in Sol right now, so thought I'd send you a PR.

  • projectserum.com RPC endpoint is no more so example in docs was failing. Make a single constant for RPC URL, explain what it is, and what values could be.
  • Minor docs tweaks (first time I ran this, I wasn't actually sure what UTR LTL or CG stood for - it's obvious in retrospect but not for newcomers)
  • Use async/await for FS operations
  • Use 2-space JS just for README (I know you use 4 for your code, but README is going to be viewed by people who write JS generally and 2-spaces is the default almost everywhere, I hope that's OK)
  • Add @types/node to devDependencies (fixes warnings using fs/promises)

 - projectserum.com RPC endpoint is no more so example in docs was failing. Make a single constant for RPC URL, explain what it is, and what values could be.
 - Minor docs tweaks (first time I ran this, I wasn't actually sure what UTR LTL or CG stood for - it's obvious in retrospect but not for newcomers)
 - Use async/await for FS operations
 - Use 2-space JS just for README (I know you use 4 for your code, but README is going to be viewed by people who write JS generally and 2-spaces is the default almost everywhere, I hope that's OK)
@skakac skakac merged commit 1bd69c0 into solflare-wallet:master Jan 23, 2023
@mikemaccana
Copy link
Contributor Author

mikemaccana commented Jan 24, 2023

Thanks @skakac ! Can you please add a LICENSE.md re: #2? I assume you want to make UTL aggregator open source but right now there's no license file.

Pick whatever license you want, but here's what popular projects use:

@mikemaccana
Copy link
Contributor Author

PS here's a PR if you just want Apache 2 like Solana, Rust, and SPL. #4

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.

2 participants