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

Rewrite: Fix 3 open bugs, repair readme syntax, consistent whitespace usage #4

Closed
wants to merge 4 commits into from

Conversation

Krinkle
Copy link

@Krinkle Krinkle commented Mar 14, 2013


Actions:

Krinkle referenced this pull request Mar 14, 2013
Fixing a security issue
* Indentation and whitespace
* Prefix global function with ef per MediaWiki convention
* Use OutputPage::setCanonicalUrl to avoid adding a second
  rel=canonical if MediaWiki core adds one already (fixes #3).
* Include patch version (".0") per semver guidelines.
* Prefer HTTPS urls for pages on servers that support it.
* Fix syntax in README.md to be proper Markdown, it currently
  looks broken when rendered on GitHub.
* Turn README.md into a symlink instead of duplicating the content.
@Krinkle
Copy link
Author

Krinkle commented Mar 14, 2013

I've updated this pull request to be rebased on top of the recent 0.2.0 release.

@Krinkle Krinkle closed this Apr 15, 2013
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

1 participant