-
Notifications
You must be signed in to change notification settings - Fork 21
Applied first batch of enhancements for 6.0 #414
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
Merged
Merged
Changes from all commits
Commits
Show all changes
9 commits
Select commit
Hold shift + click to select a range
c59529a
Adding snippet guidance Part 1
julian-cable 5cf40ab
Further enhancements to 6.0
julian-cable 8c58dca
Fixes to implement feedback
julian-cable 0932667
Further fixes
julian-cable 0b99211
Update to implement Fernando's feedback
julian-cable 4d14415
Adding addressing of issue 399 to the scope
julian-cable e3f6120
Merge branch 'dev' into jcable/6.0-enhancements
julian-cable 7c25c0e
Implemented review comments
julian-cable 4f0f09c
Resolve conflicts
julian-cable File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
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.
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.
This is not really an example of a condition. Conditions typically start with "if": "If X then Y." This example is reversing "Do X to achieve Y" to become "To achieve Y, do X."
I also think we need a better example. "Dividing storage devices into smaller chunks" could mean a variety of things, and not necessarily create partitions. Maybe have a look at https://docs.fedoraproject.org/en-US/quick-docs/creating-a-disk-partition-in-linux/
I think it'd be pretty easy to make up a "bad example" and then improve it using the commands and examples documented here.
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.
My intent here was something other than an "if ... then" construction. I've removed the entry for now, and will think further about what to do.