Add blog from Hendrik (& CAM) on Spyder 6 remote development #35
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
✅ Deploy Preview for spyder-website-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
CAM-Gerlach
force-pushed
the
blog-spyder-6-hendrik
branch
from
02e1af9 to
dab6a38
Compare
ccordoba12
reviewed
Nov 28, 2024
CAM-Gerlach
commented
Nov 29, 2024
There was a problem hiding this comment.
Thanks for the detailed review @ccordoba12 !
I applied most of the suggestions, some with further modifications, except for:
-
The ones including changing things to first person that weren't originally written that way in the outline and that @hlouzada didn't necessarily say directly himself, which I have no objection to applying so long as he gives his permission firstHe gave permission - The suggestions redundantly adding his name to the beginning of each response that are neither necessary or appropriate for this context and are already covered by your other suggested changes, as explained further in your (first) suggestion of it
- The image replacement suggestions that I was unable to apply as the direct links did not work for me
- A couple other suggestions that appeared to be misunderstandings
Thanks!
Co-authored-by: Carlos Cordoba <[email protected]>
hlouzada
reviewed
Nov 29, 2024
… per approval Co-authored-by: Carlos Cordoba <[email protected]> Co-authored-by: Hendrik <[email protected]>
|
Hendrik first person suggestions applied per @hlouzada 's permission ✔️ |
hlouzada
approved these changes
Nov 29, 2024
ccordoba12
reviewed
Nov 29, 2024
There was a problem hiding this comment.
Added the images I mentioned and responded to @CAM-Gerlach's objections about my initial suggestions.
|
I replaced the screenshots as requested and also optimized all of them to make them up to several times smaller by reducing unnecessary color depth |
ccordoba12
changed the title
CAM-Gerlach
changed the title
|
Make the requested and agreed final adjustments. Thanks @ccordoba12 (and @hlouzada , of course) for your feedback! |
CAM-Gerlach
force-pushed
the
blog-spyder-6-hendrik
branch
4 times, most recently
from
73f4198 to
0aa27b6
Compare
CAM-Gerlach
force-pushed
the
blog-spyder-6-hendrik
branch
from
0aa27b6 to
913dd42
Compare
ccordoba12
approved these changes
Nov 30, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@hlouzada and @ccordoba12 will want to review the content first