-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Wiki + Readme: MD security and improvements #9807
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
Conversation
fb17dc5 to
f9da580
Compare
f9da580 to
efca85b
Compare
doc/print_settings/calibration/adaptive-pressure-advance-calib.md
Outdated
Show resolved
Hide resolved
Noisyfox
left a comment
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 good overall. Some small issues need to be fixed.
Standardized MD GitHub Wiki format Removed outdated and malicious links Modularized calibrations Suggested calibration order added Minor bug fixes Image improvements and corrections Added winget commands Completed previous WIPs Added new WIPs Removed obsolete references Visual Changes Co-Authored-By: Noisyfox <[email protected]> Co-Authored-By: dewi-ny-je <[email protected]> Co-Authored-By: Nico Domino <[email protected]> Co-Authored-By: Martin Ulmschneider <[email protected]> Co-Authored-By: Rodrigo <[email protected]>
efca85b to
e760c8c
Compare
Noisyfox
left a comment
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.
LGTM, thanks!
|
Can someone else confirm if from the wiki clicking on the Pressure Advance guide now opens up the |
|
Yeah I noticed that too. Also images are broken too. |
Maybe the wiki builder bot? |
|
I think the links in those files have to follow sepcial rules to make it work for github wiki? |
Mm I think I know what to do. Tomorrow will try to put a new pr with it. |
Its really strange, in some cases |
|
@StdVectorBool @Noisyfox check #9858 |
Maybe use |
Will response it on new PR. |
|
NVM, I'm wrong on this one. |
|
I'm happy this got merged. How should we proceed to improve the wiki further?
|
I already have new things to do like the:
For the last one surely i will need help and thats why i was thinking the same. I will make 2 PR now (#9872 and #9873 ), one for discussion and collaboration an another for the things i already done. |

Search engines like Google can crawl and index these URLs, potentially treating them as legitimate resources. This can lead to accidental promotion of malicious sites. To avoid this, I replaced the list with a simpler CAUTION message and added clearer, more prominent official links and badges.
Description
To deprecate #9370 and utilizing the merge from #9383.