-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add simple local avatars #111
Conversation
I've changed the required version to |
NOTE: We need to wait until 2.2.0 is released before this can be merged. |
2.2 is still not released. |
2.2 still not released, however a new issue tracking release requirements is opened on 10up/simple-local-avatars#51 |
2.2 is now released, so I think this is good to be merged? |
I don't see a 2.2? |
Ah sorry I misread the comment from @adamsilverstein, it seems only the branch has been created https://github.com/10up/simple-local-avatars/tree/release/2.2.0 |
Correct, we're nearing 2.2 release and are doing some final review on adding test coverage to increase our confidence in the release. If you have opinions on 10up/simple-local-avatars#59 then by all means please comment, otherwise we're working to finalize there and get the release out. |
Thanks for the update @jeffpaul 👍 Will check it out |
@roborourke FYI that we're targeting Monday for the 2.2.0 release, apologies for the delay |
@roborourke note that the 2.2.0 release is now live |
@jeffpaul amazing, thanks for letting me know 🥇 |
closing in favor of #392 |
See #7
Assuming the package will be names
10up/simple-local-avatars
when publishedWill need an upstream fix - see humanmade/simple-local-avatars#1