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

chore: move image to github cdn #78

Merged
merged 1 commit into from
Apr 19, 2022
Merged

Conversation

will
Copy link
Contributor

@will will commented Apr 19, 2022

You can upload images straight to github and use it as a CDN instead of
having them in the repo itself. It's a small saving, but it does save
people having to download them when installing the theme.

You can upload images straight to github and use it as a CDN instead of
having them in the repo itself. It's a small saving, but it does save
people having to download them when installing the theme.
@will
Copy link
Contributor Author

will commented Apr 19, 2022

Because this is marginal feel free to reject it. Except please remove the ' from PR's if you do :)

@mvllow
Copy link
Member

mvllow commented Apr 19, 2022

Great idea using the CDN. PRs also seems more correct. Thanks 💜

@mvllow mvllow merged commit e7f17f7 into rose-pine:main Apr 19, 2022
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