-
-
Notifications
You must be signed in to change notification settings - Fork 16
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
Dist package 1.1.2 has two src folders #15
Comments
Unfortunately, that's out of our control; the dist files are auto-generated by github. If you look at the tree for that tag (https://github.com/zfcampus/zf-oauth2/tree/d2545896b0c94f31db8111790763ef27cbdef1ad), the directory duplication is not present. I can try and raise an issue with @github if necessary. Originally posted by @weierophinney at zfcampus/zf-oauth2#99 (comment) |
Hi, thanks @weierophinney for reply. Maybe you could try release another version (without any changes) - probably it was temporary issue? Originally posted by @kusmierz at zfcampus/zf-oauth2#99 (comment) |
@kusmierz Releases are tied to tags, and Composer compares the tag to the specified sha1. As such, re-tagging is a huge pain, and since we cannot be guaranteed that the snafu won't repeat, one that's not worthwhile, especially considering this is the only such report. Originally posted by @weierophinney at zfcampus/zf-oauth2#99 (comment) |
+1 Originally posted by @Kyniu at zfcampus/zf-oauth2#99 (comment) |
When using composer and
--prefer-dist
option, composer will download zip file.Exactly this one: https://packages.zendframework.com/composer/zfcampus-zf-oauth2-d2545896b0c94f31db8111790763ef27cbdef1ad-zip-bb2d0d.zip It has two src dirs inside:
because of it, composer's optimizer shows some warnings:
Nothing critical in fact, but looks bad, could you fix it?
Originally posted by @kusmierz at zfcampus/zf-oauth2#99
The text was updated successfully, but these errors were encountered: