Skip to content

Conversation

mathieu-aubin
Copy link
Contributor

No description provided.

@ericpruitt ericpruitt changed the title version bump Upgrade NGINX from v1.19.8 to v1.19.10 Apr 14, 2021
@ericpruitt ericpruitt merged commit 65386c2 into ericpruitt:master Apr 14, 2021
@ericpruitt
Copy link
Owner

Thanks for submitting this diff.

@mathieu-aubin
Copy link
Contributor Author

i have made the pull a bit too fast maybe, not thinking about the patch and not having tested. This is pretty 'green' on my part. Hopefuly things still work as intended...

@ericpruitt
Copy link
Owner

I tested the patch locally before I merged it.

@ericpruitt
Copy link
Owner

I know rewriting history is generally frowned upon, but merging this commit from the GitHub web UI didn't work as intended, and I don't like the way the commits were done. I'm going to reset the repo and apply the change from the command line and force-push the updated repo, so you won't be able do a standard pull since the repos will have diverged.

@ericpruitt
Copy link
Owner

Done. HEAD is now 91e3da2 which includes a Make target for testing the NGINX binary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants