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

(feat): allow sl pr submit to support organization-specific GitHub URLs #1021

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

vincent-4
Copy link

@vincent-4 vincent-4 commented Jan 30, 2025

Example usage:
ssh://[email protected]/bolinfest/escoria-demo-game.git set as the default URL will work now because of an updated regex pattern. Updated docstring and test for this function.
References #1015

[email protected]/foo/bar.git

- updated regex pattern in
`eden/scm/sapling/ext/github/github_repo_util.py`
- has the same structure validation
- added docstring validation
@facebook-github-bot
Copy link
Contributor

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. (Because this pull request was imported automatically, there will not be any future comments.)

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

Successfully merging this pull request may close these issues.

2 participants