Skip to content

change non-existent links to existing links in every file #3451

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

BlueButterflies
Copy link
Contributor

Hi guys, I changed the non-existent links with existing links. These changes are necessary for the correct learning of the examples of the open source contribution.

@BlueButterflies BlueButterflies requested a review from a team as a code owner June 9, 2025 15:10
@ahpook
Copy link
Contributor

ahpook commented Jun 13, 2025

Hi @BlueButterflies ! Thanks for another great contribution!
Question: this has a lot of the same changes as #3452 - is that on purpose? I think we can only take one of them but it's hard to tell which is correct.

@BlueButterflies
Copy link
Contributor Author

Hi @ahpook , If #3451 and #3452 seem the same, they are not, because they have two more different fail files to add to the site. #3451 has "Security best practices for you project" file in Italian language and #3452 has "Security best practices for you project" file in Bulgarian language. I worked on two different branches, one is #3451 where it contains the fail "Security best practices for you project" in Italian language and the correction of the links and more and #3452 where it should contain only the "Security best practices for you project" file in Bulgarian language. I don't know why in the branch with the fail "Security best practices for you project" in Bulgarian language the link changes were also added.

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.

2 participants