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

Upgraded to elm 0.19#6

Open
savardd wants to merge 5 commits intoelm-community:masterfrom
savardd:master
Open

Upgraded to elm 0.19#6
savardd wants to merge 5 commits intoelm-community:masterfrom
savardd:master

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