-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add datashuttle blog #98
Conversation
Yeah I would just use the light mode image here. It looks nicer to have an image with a white background in dark mode, than an invisible image in lightmode. Also, I suspect most people will view the website in light mode? I use dark mode for most things, but not stuff like this. |
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.
Looks nice @JoeZiminski, I've added a few comments.
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.
Any way you can put your name first @JoeZiminski ? You did most of the work on datashuttle and on this blogpost.
i've left a few comments, mostly about phrasing.
Co-authored-by: Adam Tyson <[email protected]>
Co-authored-by: Adam Tyson <[email protected]>
Co-authored-by: Adam Tyson <[email protected]>
Co-authored-by: Adam Tyson <[email protected]>
Co-authored-by: Adam Tyson <[email protected]>
a265ad9
to
bd322e5
Compare
Hey finished with some final tidy ups and tweaks. Could tweak it forever! Would be great if someone can have a final read-through then happy for it to be merged next week. |
docs/source/blog/datashuttle.md
Outdated
There are a number of shortcuts to ensure only | ||
information relevant to your specific project must be typed. |
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.
I'm not sure what this sentence means. Should it read "only be typed" at the end?
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.
I also coudn't parse this sentence. I'll rewrite it as:
"There are a number of shortcuts to reduce the amount of manual typing."
Co-authored-by: Adam Tyson <[email protected]>
Hey @adamltyson, I did a final pass on this post, caught a few more typos and formatting errors, and very slightly rephrased some sentences (ensuring, for example, that data are consistently plural). I also cross-linked the two blogposts in their introductions. I'm merging this now and we can start sharing the blog posts tomorrow. |
This blog is the companion piece to #96. It's aims are to introduce datashuttle as a tool, giving an overview of the benefits of using it and its features.