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

Use HTTPS in link to Wikipedia article on password strength #1164

Merged
merged 1 commit into from
Jan 7, 2025

Conversation

scottdotweb
Copy link
Contributor

The link to the Wikipedia article was added here in 2008, and Wikipedia went HTTPS-only in 2015. (https://diff.wikimedia.org/2015/06/12/securing-wikimedia-sites-with-https/)

@ikerexxe
Copy link
Collaborator

ikerexxe commented Jan 7, 2025

The changes themselves look good, but the commit message does not follow the project practices.

We usually limit the commit title to 50 characters and the message lines to 72. In addition, we mention the folder in which we are committing the changes in the title and add the links at the end. Here is an example to show you what is expected: 0b8c0c8

@scottdotweb scottdotweb force-pushed the scottdotweb-patch-1 branch 2 times, most recently from e398000 to 674cf1d Compare January 7, 2025 10:59
The link to Wikipedia's article "Password strength" was added here in
2008 and Wikipedia went HTTPS-only 7 years later.

Link: <https://diff.wikimedia.org/2015/06/12/securing-wikimedia-sites-with-https/>
@scottdotweb
Copy link
Contributor Author

Hi Iker and thanks for the guidance 🙂 This project is quite outside my usual area of contributions. I've modified the commit per those rules.

Copy link
Collaborator

@ikerexxe ikerexxe left a comment

Choose a reason for hiding this comment

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

LGTM!

CodeQL seems stuck and the changes aren't code related, so I'm merging it.

@ikerexxe ikerexxe merged commit cc2ef99 into shadow-maint:master Jan 7, 2025
9 checks passed
@scottdotweb scottdotweb deleted the scottdotweb-patch-1 branch January 7, 2025 15:41
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.

3 participants