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

Disable link to seed-rs.org due to security concern #127

Merged
merged 1 commit into from
Jan 3, 2025
Merged

Disable link to seed-rs.org due to security concern #127

merged 1 commit into from
Jan 3, 2025

Conversation

Rindrics
Copy link
Contributor

@Rindrics
Copy link
Contributor Author

FIY @akhilman (as org member)

akhilman
akhilman approved these changes Dec 31, 2024
View reviewed changes
Copy link
Member

@akhilman akhilman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you. Maybe we should remove this link completely or replace it with the github page.

@akhilman
Copy link
Member

Seems like I don't have write access or I'm not smart enough and could not figure out how to merge this commit.

@Rindrics
Copy link
Contributor Author

Rindrics commented Jan 2, 2025

Thank you for checking this out @akhilman

@MartinKavik (I think you have write permission to this repo) Would you merge this or disable the link with another way?

@MartinKavik
Copy link
Member

Hi, I've switched @akhilman role from member to org owner

@akhilman akhilman merged commit 25d1424 into seed-rs:master Jan 3, 2025
@akhilman
Copy link
Member

akhilman commented Jan 3, 2025

I will run through all org's projects and will remove links to broken site.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@akhilman akhilman akhilman approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Rindrics @akhilman @MartinKavik