Skip to content
This repository has been archived by the owner on Feb 1, 2021. It is now read-only.

SSH URL Update #105

Closed
wants to merge 7 commits into from
Closed

SSH URL Update #105

wants to merge 7 commits into from

Conversation

quantum-x
Copy link

Hi guys.

  • Small tweak to allow SSH URLS to be added.
  • Accepts GH and non GH SSH URLS
  • Conforms with Sismo's formatting for @Branch

IE:
$projects[] = new Sismo\GithubProject('Project Dev Branch', 'git@github.com:people/project.git@dev', $notifier);
or just as happily
$projects[] = new Sismo\GithubProject('Project Dev Branch', 'user@othersite.com:people/project.git', $notifier);

First PR, please be gentle.

@@ -38,6 +38,9 @@ public function setRepository($url)
} elseif (preg_match('#^[a-z0-9_-]+/[a-z0-9_-]+$#i', $this->getRepository())) {
$this->setUrlPattern(sprintf('https://github.com/%s/commit/%%commit%%', $this->getRepository()));
parent::setRepository(sprintf('https://github.com/%s.git', $this->getRepository()));
} elseif (preg_match('#^git@github.com:[a-z0-9_-]+/[a-z0-9_-]+\.git@?([a-z0-9_-]+)?$#i', $url)) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add test for that ?

@Mihailoff
Copy link

Vote for this 👍

@lyrixx
Copy link
Member

lyrixx commented Dec 1, 2013

ping @quantum-x

@quantum-x
Copy link
Author

Hi guys.
Sorry, for the delay, I've had other priorities. Added to my to-do list for
tomorrow.
Thanks for the reminder!

On Sun, Dec 1, 2013 at 4:42 PM, Grégoire Pineau notifications@github.comwrote:

ping @quantum-x https://github.com/quantum-x


Reply to this email directly or view it on GitHubhttps://github.com//pull/105#issuecomment-29575910
.

@h4cc
Copy link

h4cc commented Jan 13, 2014

I had to patch this for my local installation. Also vote for this. 👍

@quantum-x
Copy link
Author

Apologies for the delay - I am incredibly swamped with real-life.
@lyrixx - I've updated the tests, and should have tweaked the code according to PSR1 / 2.

If I've missed something, I'm going to have to pass the baton to someone else; not sure I have the time unfortunately. Fingers crossed that all is good.

@lyrixx
Copy link
Member

lyrixx commented Feb 6, 2014

Test does no pass on travis.

@quantum-x
Copy link
Author

C'est bien possible que je l'ai mal fait.
Je suis vraiment désolé - je ne suis pas du tout doué en PHPUnit :-/

On Thu, Feb 6, 2014 at 5:24 PM, Grégoire Pineau notifications@github.comwrote:

Test does no pass on travis.

Reply to this email directly or view it on GitHubhttps://github.com//pull/105#issuecomment-34340769
.

@lyrixx
Copy link
Member

lyrixx commented Jul 22, 2015

Closing as there is no update. Feel free to re-open.

@lyrixx lyrixx closed this Jul 22, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants