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

Revert pull 707 - interviewbit spam #711

Merged
merged 1 commit into from
Dec 1, 2022

Conversation

6d64
Copy link
Contributor

@6d64 6d64 commented Nov 30, 2022

This reverts pull request #707 - I believe it added a link to commercial marketing materials.

User @ASAlishaa is just adding links to interviewbit.com to various projects' documentation, always with the same wording, same commit message, same branch name, just different articles all on the same tutorial website that promotes paid online programming courses.

Here is some of that user's commit history as proof, this is about a third of the repositories that they have contributed to in the same manner:

1 2 3 4 5 6 7 8 9 10 11 12 13 14

Same links archived in case they become unavailable:
1 2 3 4 5 6 7 8 9 10 11 12 13 14

Reverted commit that was added by a bot adding interviewbit spam to
repos on github
@6d64
Copy link
Contributor Author

6d64 commented Nov 30, 2022

FYI I reported the issue to Github as well, and this user's history is no longer available and the original pull request #707 has been removed as well. The archived links will still work.

@stleary
Copy link
Owner

stleary commented Dec 1, 2022

Interesting. @6d64 Thanks for catching this.

@stleary
Copy link
Owner

stleary commented Dec 1, 2022

What problem does this code solve?
Revert some doc spam

Risks
Low

Changes to the API?
N/A

Will this require a new release?
No

Should the documentation be updated?
No

Does it break the unit tests?
N/A

Was any code refactored in this commit?
N/A

Review status
APPROVED

To be merged immediately.

@stleary stleary merged commit 5920eca into stleary:master Dec 1, 2022
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