Skip to content
This repository has been archived by the owner on Mar 13, 2019. It is now read-only.

Fix #113: use the "community" repo for php7-pear in Alpine-based image #114

Closed
wants to merge 1 commit into from
Closed

Fix #113: use the "community" repo for php7-pear in Alpine-based image #114

wants to merge 1 commit into from

Conversation

EvgenyOrekhov
Copy link

No description provided.

@BastienM
Copy link
Contributor

Hi,

Already patched the issue last week, see #110 ;)

@EvgenyOrekhov
Copy link
Author

@BastienM indeed. I can see now that your build failed because of 500 Internal Server Error. You should've force-pushed your commits to trigger a new CI build and make your PR green.

@BastienM
Copy link
Contributor

Considering the 500, I didn't really thought that it would matter.
It should be ok by now.

Now let's hope the team will merge either one the patch soon :)

@EvgenyOrekhov
Copy link
Author

@BastienM some maintainers might not even look at a PR that has failed checks, they will wait until a contributor makes it green.

@RobLoach
Copy link
Owner

RobLoach commented Nov 2, 2016

Thanks a lot! I see it's @edge instead of @community. That doesn't matter so much, correct? Other then that I think this can be closed.

Thanks for the help, @EvgenyOrekhov, and the ping in re-running tests. Well done, you two!

@RobLoach RobLoach closed this Nov 2, 2016
@EvgenyOrekhov
Copy link
Author

@RobLoach @edge is just a user-defined alias for the repo link, it doesn't really matter.
You're welcome! I think you can close #106 and #113 now =)

@BastienM
Copy link
Contributor

BastienM commented Nov 2, 2016

@RobLoach : as @EvgenyOrekhov said, it just a arbitrary tag.

The package still is in the edge repositories but has moved from testing to community staging :)

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.

3 participants