Dead links in README.md #15455
Description
Activity
Our linting ado job also failing because of this, we have to address this before 0.82 releases
If you look at the commit history, the two files were removed as part of a PR #15370 due to the documentation generated with the help of AI, and it having no relevance with RNW and it's related secruity info. There are either two fixes to this,
- Generate the documents for this, without AI slop
- Remove these links entirely from the README.md
One thing that can be done alternatively is to add these files and add a sort of "TODO" to it.
Since this is tagged as a "good first issue", can I contribute to this? Would love to contribute to the documentation fix for RNW!
Kushal SM (@mrkushalsm) Yes you can raise the PR for contribution..
If you look at the commit history, the two files were removed as part of a PR #15370 due to the documentation generated with the help of AI, and it having no relevance with RNW and it's related secruity info. There are either two fixes to this,
- Generate the documents for this, without AI slop
- Remove these links entirely from the README.md
One thing that can be done alternatively is to add these files and add a sort of "TODO" to it.
Since this is tagged as a "good first issue", can I contribute to this? Would love to contribute to the documentation fix for RNW!
Use the 2nd one, remove the link from the ReadMe
Thank you Vineeth (@vineethkuttan) for assiging this issue to me.
Just wanted to clear one thing going ahead with the 2nd fix I mentioned, so do I,- Still keep the entire section of "Security Documentation" along with it's content and just remove the dead links
- Just remove the entire section of "Security Documentation" along with it's content
In the meantime I shall fork the repository and keep my changes ready for a PR, but would love your input on this!
Also this is my first time contributing, so I had one more doubt, do I create a branch and push my changes there and send a PR? And if so shall I go with the naming system user/kushalsm/fix-15455 OR I directly send my PR to the main branch?
Thank you again for helping me out!
Just remove the entire section of "Security Documentation" along with it's content
Remove it please, without readme it won't make any sense.
Refer to these two docs and online resources to contribute by raising a PR
I have raised a PR docs: Remove outdated security documentation from README.md #15511
Please review it!
Reacted by Vineeth
At the Security Documentation section of the README.md, both the links for Security Configuration Guide and Security Best Practices lead to a file that has been deleted (or moved).