Skip to content
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

Update to php7.1.27, 7.2.16 and 7.3.3. #55

Merged
merged 1 commit into from
Apr 1, 2019

Conversation

falkzoll
Copy link
Contributor

Update to actionloop 42fd5e6.
Copy link
Member

@akrabat akrabat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, assuming Travis passes.


## 1.12.0-incubating
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Curious about 1.12.0-incubating. Are there no changes in that release?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added the tag since I have seen this in the docker registry (e.g. https://hub.docker.com/r/openwhisk/action-php-v7.1/tags) and it was not documented in the CHANGELOG.md of the individual runtime images, yet.

I looked at the history.
The change from 1.0.3 to 1.12.0-incubating was to revert the content of copyright to the content in the template in LICENSE.txt (93c1871) on Aug 28, 2018. This change is not related to the generated docker images.
Means, the images of 1.0.3 and 1.12.0-incubating are functional identical. Which I tried to visualize by not having a 'Changes' section in between the 2 versions.

In case there is a more common way to show this, I can adjust it accordingly :-).

@falkzoll
Copy link
Contributor Author

falkzoll commented Apr 1, 2019

up to now, I do not have the permission to merge in these repos, can one of the committers do the merge?
Thanks in advance :-).

@dgrove-oss dgrove-oss merged commit 9c5d3d7 into apache:master Apr 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants