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

Upgraded to elm 0.19 #6

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

savardd
Copy link

@savardd savardd commented Aug 27, 2019

Hello,
I did upgrade these icons to elm 0.19.
Can you review and merge the change so all can benefit from it?
thanks,
Daniel

@mgold
Copy link
Member

mgold commented Aug 27, 2019

See this comment. Is this PR still necessary?

@savardd
Copy link
Author

savardd commented Aug 27, 2019

hello mgold,
Effectivly. This comment came after I started upgrading... This PR is not really necessary but could help those that find your project to upgrade. Maybe, instead, you can add the URL of the comment of '4onen' in the Readme.md?
Thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants