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.

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

Fix a typo link to a malicious site #234

Closed
wants to merge 2 commits into from
Closed

Conversation

j824h
Copy link

@j824h j824h commented Dec 20, 2023

The target link of the image is set incorrectly.
Currently, the link leads to a website that seems to spam by redirection.
Since this is a security risk with a simple fix, I chose to skip the procedure.

dimtsap and others added 2 commits November 27, 2023 13:15
Removes seaborn from example dependencies
The target link of the image is set incorrectly.

Currently, the link leads to a website that seems to spam by redirection.
@dimtsap
Copy link
Contributor

dimtsap commented Dec 20, 2023

Can you please create the pull request for the Developement branch so that I can merge it when ready.?

@j824h j824h changed the base branch from master to Development December 21, 2023 10:24
@j824h
Copy link
Author

j824h commented Jan 2, 2024

@dimtsap Thanks for the reply! The target branch has been set again as you instructed. Hope it helps.

dimtsap added a commit that referenced this pull request May 9, 2024
@dimtsap dimtsap closed this May 20, 2024
@j824h j824h deleted the patch-1 branch May 21, 2024 06:28
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