Refactoring Sealights and AppDynamics Hooks

cloudfoundry / python-buildpack

Cloud Foundry buildpack for the Python Language

http://docs.cloudfoundry.org/buildpacks/

Apache License 2.0

119 stars 277 forks source link

Closed liornabat-sealights closed 1 year ago

liornabat-sealights commented 1 year ago

Thanks for contributing to the buildpack. To speed up the process of reviewing your pull request please provide us with:

A short explanation of the proposed change: This PR refactor PR 809 per @robdimsdale request
An explanation of the use cases your change solves
[ x] I have viewed signed and have submitted the Contributor License Agreement
[ x] I have made this pull request to the develop branch
[x ] I have added an integration test

robdimsdale commented 1 year ago

@liornabat-sealights thank you for submitting this so quickly! and for fixing the app dynamics integration test too. I really appreciate it.

brayanhenao commented 1 year ago

Any update @liornabat-sealights? We need to cut a release asap and we want these changes to be in that release.

robdimsdale commented 1 year ago

I fixed the last remaining typo. I'll enable auto-merge, and we can release once the PR is merged.