-
Notifications
You must be signed in to change notification settings - Fork 29
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
Scaling on rotation blog post #182
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just pushed my iteration, lgtm! 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you look over these please & incorporate them as you see fit?
Just pushed another iteration, addressing most of your comments :) Let me know if there's anything you think needs changing and I'd be happy to take another swing at this until we get it perfect |
@Missing-Tech, would you be able to take it from here, wrap up the blog post & fix the linters & get it merged? |
Sure I'd love to! |
Sorry for the delay here! Just fixed some linting errors and it should be merged soon :) |
@arturi Is this ok to be merged? |
* main: Scaling on rotation blog post (#182) deps: update uppy to 3.20.0 (#190) Workaround setup-node Yarn bug (#189) Fix broken link meta: upgrade to Yarn 4 (#188) Upgrade to Docusaurus V3 (#184) Companion: document new arguments for s3.bucket and s3.getKey (#183) Document S3 endpoint for Companion (#179) deps: update uppy to 3.18.1 (#180) Add "Custom stores" guides (#172) deps: update uppy to 3.18.0 (#177)
No description provided.