-
Notifications
You must be signed in to change notification settings - Fork 230
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
feat: support proxy configuration #621
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
action.yml
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🫡
Hi @chingor13, I was wondering if there is a chance this one is given some attention? There is a corresponding change already merged in the https://github.com/googleapis/release-please and it's causing a bit of discrepancy between the two. |
I'm unable to resolve the conflict in the PR. This PR needs to revert the conflicting files. |
@chingor13 , I have resolved the conflict. |
With our recent efforts to the upstream google release please package (googleapis/release-please#1639 and googleapis/release-please#1655) We are now able to configure the proxy settings in an easy way.
This PR exposes the proxy configuration to the action users and is helpful for users who run the action behind a proxy server.
This PR will resolve #458 and #629 as well