a more complete guide to dressing a hammer .. I hope#35
Open
bitbasher wants to merge 2 commits intoakatch:mainfrom
Open
a more complete guide to dressing a hammer .. I hope#35bitbasher wants to merge 2 commits intoakatch:mainfrom
bitbasher wants to merge 2 commits intoakatch:mainfrom
Conversation
I saw (too late) that my update is duplicating the intent of the last two sections should I move my content to the end as you do .. OR just remove the last 2 sections
I still cannot find any documentation on the {{}} markup so I hope I am using it correctly.
Feedback on the whether I should use the procedure sections at the end of the page, or not, would be good .. I like having the text flow from "cause of problem" to "fix for problem" all in a block, but if Hammerbros style wants it to stay as is I can move paragraphs around to fit it
Author
|
oh right .. I think the two warning sections a the start of the page should be Either made into a single block (as I have hopefully done with the /br tag .. but I am doubting that it will work. I cannot see the end result without making a server and I cannot do that at this time |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
You will see that I am proposing a wider scope for this how-to item. Big red flag for me was the apparent recommendation to use a grinder on a hammer is "fraught with peril" and I think it important to mention some of the other standard blacksmith tools that can stand to be dressed.
One possible issue is my deviation from having Procedure sections last in the page .. I have done it all in a block and wonder if I should remove those trailing sections. Feedback on this would be appreciated