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 correctly sized gravatars when possible #1206

Merged
merged 1 commit into from May 27, 2014

Conversation

haarg
Copy link
Member

@haarg haarg commented May 24, 2014

Provides a gravatar_fixup vmethod that accepts a pixel size to set on
gravatar URLs. It also adjusts them to use HTTPS, replacing the
existing repeated templated code.

Provides a gravatar_fixup vmethod that accepts a pixel size to set on
gravatar URLs.  It also adjusts them to use HTTPS, replacing the
existing repeated templated code.
@oalders
Copy link
Member

oalders commented May 24, 2014

I see the test suite isn't passing. One problem is that the Dist::Zilla search has been hijacked by Zilla::Dist https://metacpan.org/search?q=dist%3A%3Azilla :)

@haarg
Copy link
Member Author

haarg commented May 24, 2014

There were some earlier test failures that were real, but the remaining ones are not related to this PR.

@tsibley
Copy link
Contributor

tsibley commented May 25, 2014

@oalders For the "dist zilla" query that's used, Zilla::Dist is scored slightly by ES higher than Dist::Zilla, presumably because the words "dist" and "zilla" are found slightly more frequently or something. The request and response are here: https://gist.github.com/tsibley/863fc6ae99cd2bce151c

Searching for "dist::zilla" has the same behaviour, but "Dist::Zilla" works as expected because Dist::Zilla gets a score boost from an exact package name match.

@oalders
Copy link
Member

oalders commented May 27, 2014

Thanks! 👍

oalders added a commit that referenced this pull request May 27, 2014
Use correctly sized gravatars when possible
@oalders oalders merged commit ef7d4f2 into metacpan:master May 27, 2014
oiami pushed a commit to oiami/metacpan-web that referenced this pull request Jun 9, 2014
Use correctly sized gravatars when possible
oiami pushed a commit to oiami/metacpan-web that referenced this pull request Jun 9, 2014
Use correctly sized gravatars when possible
@haarg haarg deleted the gravatar-fixup branch October 12, 2014 12: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.

None yet

3 participants