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

anonymize the docs #355

Merged
merged 1 commit into from
Aug 24, 2023
Merged

anonymize the docs #355

merged 1 commit into from
Aug 24, 2023

Conversation

adamjanovsky
Copy link
Collaborator

No description provided.

@adamjanovsky
Copy link
Collaborator Author

adamjanovsky commented Aug 24, 2023

@J08nY I've anonymized the docs. The problem is that they link the markdown files from the repository, so I needed to anonymize them as well. I just dropped the mention of authors from the GitHub readme. I suppose we can live with that for two months. People will still see the maintainers and can open an issue or mail the maintainers directly.

Also, it's not perfectly anonymous, some badges e.g., lead to PyPi with authors, etc. But we likely don't need to go that far.

It should suffice to revert this commit for deanonymization.

@J08nY
Copy link
Member

J08nY commented Aug 24, 2023

Agreed.

@adamjanovsky adamjanovsky merged commit 381a89f into main Aug 24, 2023
5 of 6 checks passed
@adamjanovsky adamjanovsky deleted the anonymize-docs branch August 24, 2023 09:01
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.

None yet

2 participants