-
-
Notifications
You must be signed in to change notification settings - Fork 424
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 github server url for pushing to fork #1315
Conversation
@MildC Thank you for this contribution! Your fix looks good. I'm just going to merge it to a feature branch in this repository to fix the build before merging to Have you been able to test your feature branch and confirm that it works? It might be difficult for me to do that because I don't run self-hosted GitHub. |
@peter-evans Thanks for merging! I tested the new code on the feature branch and it can create the PR properly! |
@MildC Great. Thank you for confirming! |
* feat: support github server url for pushing to fork (#1315) Co-authored-by: Kevin Zhu <[email protected]> * fix: code formatting * test: fix tests for getRemoteUrl Co-authored-by: MildC <[email protected]> Co-authored-by: Kevin Zhu <[email protected]>
Released as Thank you! |
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [peter-evans/create-pull-request](https://github.com/peter-evans/create-pull-request) | action | patch | `v4.2.1` -> `v4.2.4` | --- ### Release Notes <details> <summary>peter-evans/create-pull-request</summary> ### [`v4.2.4`](https://github.com/peter-evans/create-pull-request/releases/tag/v4.2.4): Create Pull Request v4.2.4 [Compare Source](https://github.com/peter-evans/create-pull-request/compare/v4.2.3...v4.2.4) ⚙️ Patches some recent security vulnerabilities. ##### What's Changed - Update concepts-guidelines.md by [@​chrisbruford](https://github.com/chrisbruford) in [https://github.com/peter-evans/create-pull-request/pull/1610](https://github.com/peter-evans/create-pull-request/pull/1610) - 58 dependency updates by [@​dependabot](https://github.com/dependabot) ##### New Contributors - [@​chrisbruford](https://github.com/chrisbruford) made their first contribution in [https://github.com/peter-evans/create-pull-request/pull/1610](https://github.com/peter-evans/create-pull-request/pull/1610) **Full Changelog**: peter-evans/create-pull-request@v4.2.3...v4.2.4 ### [`v4.2.3`](https://github.com/peter-evans/create-pull-request/releases/tag/v4.2.3): Create Pull Request v4.2.3 [Compare Source](https://github.com/peter-evans/create-pull-request/compare/v4.2.2...v4.2.3) ##### What's Changed - fix: add check for missing token input by [@​peter-evans](https://github.com/peter-evans) in [https://github.com/peter-evans/create-pull-request/pull/1324](https://github.com/peter-evans/create-pull-request/pull/1324) **Full Changelog**: peter-evans/create-pull-request@v4.2.2...v4.2.3 ### [`v4.2.2`](https://github.com/peter-evans/create-pull-request/releases/tag/v4.2.2): Create Pull Request v4.2.2 [Compare Source](https://github.com/peter-evans/create-pull-request/compare/v4.2.1...v4.2.2) ##### What's Changed - fix: support github server url for pushing to fork by [@​peter-evans](https://github.com/peter-evans) in [https://github.com/peter-evans/create-pull-request/pull/1318](https://github.com/peter-evans/create-pull-request/pull/1318) ##### New Contributors - [@​kevinzhu-sa](https://github.com/kevinzhu-sa) made their first contribution in [https://github.com/peter-evans/create-pull-request/pull/1315](https://github.com/peter-evans/create-pull-request/pull/1315) **Full Changelog**: peter-evans/create-pull-request@v4.2.1...v4.2.2 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/levaintech/keychain). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNS4xMzEuMCIsInVwZGF0ZWRJblZlciI6IjM1LjEzMS4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* feat: support github server url for pushing to fork (peter-evans#1315) Co-authored-by: Kevin Zhu <[email protected]> * fix: code formatting * test: fix tests for getRemoteUrl Co-authored-by: MildC <[email protected]> Co-authored-by: Kevin Zhu <[email protected]>
This commit added the support for alternative github server but the server url is still hard coded in
getRemoteUrl
function. This breaks the pushing to fork workflow