Skip to content
This repository has been archived by the owner on Feb 9, 2023. It is now read-only.

Update logo URL #39

Merged
merged 1 commit into from
Jul 3, 2020
Merged

Update logo URL #39

merged 1 commit into from
Jul 3, 2020

Conversation

abraham
Copy link

@abraham abraham commented May 18, 2020

Which issue, if any, is this issue related to?

npm shows "not secure" in Chrome when loading http images on https sites. https://www.npmjs.com/package/@stylelint/postcss-css-in-js

Screen Shot 2020-05-18 at 11 38 09

Is there anything in the PR that needs further explanation?

It wasn't an issue on GitHub as they proxy image assets.

Copy link
Member

@mattxwang mattxwang left a comment

Choose a reason for hiding this comment

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

Looks like there hasn't been much response on this, but I'm surprised that this is a problem! Anyways, the old URL redirects to this new one, so I don't see a problem with merging. Thank you for the contribution, LGTM!

@mattxwang mattxwang merged commit d02dcc4 into stylelint:master Jul 3, 2020
mattxwang added a commit that referenced this pull request Jul 3, 2020
@mattxwang
Copy link
Member

@jeddy3 any chance we could update the NPM listing with this now? Not sure exactly how the README page is generated.

@jeddy3
Copy link
Member

jeddy3 commented Jul 12, 2020

Not sure exactly how the README page is generated.

It comes from the published package. I just did a release, so it should be fixed now.

@abraham abraham deleted the patch-1 branch July 12, 2020 14:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants