Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix: Update broken links all secDevLabs #588 #597

Open
wants to merge 32 commits into
base: master
Choose a base branch
from
Open

fix: Update broken links all secDevLabs #588 #597

wants to merge 32 commits into from

Conversation

fabiohcnobre
Copy link

@fabiohcnobre fabiohcnobre commented Oct 13, 2022 •
edited
Loading

I looked at the whole project and checked the broken links.

In case I left a link that I couldn't verify.
Please let me know that I merge with this PR.

NOTE I HAVE LOOKED AT THE ENTIRE GITHUB PROJECT

#588

@fguisso
Copy link
Member

fguisso commented Oct 18, 2022

Awesome @fabiohcnobre your PR is valid now. You forget the Solutions Links as we said in issue #588. Anyway, we accept this contribution.

@fabiohcnobre
Copy link
Author

fabiohcnobre commented Oct 19, 2022 •
edited
Loading

Where are the other links? I can look too.

@fabiohcnobre
Copy link
Author

I looked at the other files and adjusted the link so that it shows at the end of the file.

@henriporto henriporto self-assigned this Nov 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@henriporto henriporto henriporto left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@henriporto henriporto

Labels
hacktoberfest-accepted
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fabiohcnobre @fguisso @henriporto