Skip to content

Conversation

@juanotto
Copy link
Contributor

@juanotto juanotto commented Oct 28, 2025

Several improvements:

  • Updated all the images with screenshots using NB 27 on Ubuntu.
  • Old images removed (they were only used here).
  • Format using asciidoc to keep elements inside list items for steps with instructions.
  • Text simplification, kept the general structure but modified the text quite a bit.

@juanotto juanotto marked this pull request as ready for review October 28, 2025 10:31
@juanotto
Copy link
Contributor Author

@mbien @ebarboni have a look when you can. I applied the same types of changes as in the previous docs PR, but a lot more editing on the text for this one.

@mbien mbien self-requested a review October 28, 2025 14:58
Copy link
Member

@mbien mbien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Text simplification, kept the general structure but modified the text quite a bit.

i like the streamlining! Reads good IMO.

left a few suggestions inline.

Copy link
Member

@mbien mbien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reads good now I think

@mbien
Copy link
Member

mbien commented Nov 3, 2025

if you think this is done, feel free to squash so that we can get this update in too

 - Update images with NB27
 - Improve lists format with continuations (+)
 - Text simplified
 - Limit line to 120 characters
@mbien mbien merged commit 6b288e4 into apache:main Nov 5, 2025
@juanotto juanotto deleted the update-sel-part1 branch November 11, 2025 06:34
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.

3 participants