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

Github Enterprise Compatibility #42

Closed
markkevans opened this issue Sep 28, 2013 · 7 comments
Closed

Github Enterprise Compatibility #42

markkevans opened this issue Sep 28, 2013 · 7 comments
Labels

Comments

@markkevans
Copy link

I couldn't find a way to use this against a github enterprise instance as it always attempts to connect to github.com / api.github.com

This is possible according to the documentation at http://peter-murach.github.io/github/

github = Github.new do |config|
  config.endpoint    = 'https://github.company.com/api/v3'
  config.site        = 'https://github.company.com'
end
codenamev pushed a commit that referenced this issue Oct 16, 2013
Closes #46, #41, #42, #43, #44

LGTM given by: @nhance

Squashed commit of the following:

commit 0e10c209bde4be993c59e2b0fce68982a1bb2f16
Author: Valentino <[email protected]>
Date:   Wed Oct 16 08:35:37 2013 -0400

    Update fetching of comments for new API

commit 0457f72
Author: Valentino <[email protected]>
Date:   Wed Oct 16 08:15:54 2013 -0400

    Include pull request title in squashed merge message

commit 99e1dd6
Author: Valentino <[email protected]>
Date:   Wed Oct 16 08:11:24 2013 -0400

    Cleanup duplicate content in readme, and add a LGTM option

commit 51c3406
Author: Valentino <[email protected]>
Date:   Tue Oct 15 15:16:55 2013 -0400

    [fix] Update to github_api gem caused issue merging comments

commit 6822f78
Author: Valentino <[email protected]>
Date:   Tue Oct 15 15:07:52 2013 -0400

    Updated Readme for new setup command options for local and enterprise

commit 8a79ffd
Author: Valentino <[email protected]>
Date:   Tue Oct 15 11:07:39 2013 -0400

    Initial additions to setup command for project-specific and enterprise

commit 1a15364
Author: Valentino <[email protected]>
Date:   Tue Oct 15 08:36:00 2013 -0400

    Refactor for pull request comments

commit eb16110
Author: Valentino <[email protected]>
Date:   Tue Oct 15 08:11:59 2013 -0400

    [fix] Review flags not being passed

commit d2097cc
Author: Valentino <[email protected]>
Date:   Thu Oct 10 10:37:48 2013 -0400

    [fix] need to move review flags into scope of command

commit 7fa5e39
Author: Valentino <[email protected]>
Date:   Thu Oct 10 10:17:26 2013 -0400

    Don't yet upgrade github_api gem

commit 3a9dca0
Author: Valentino <[email protected]>
Date:   Thu Oct 10 09:09:50 2013 -0400

    Updated readme with instructions on setting up GitHub Enterprise accounts

commit 7842156
Merge: aec25db a8ca369
Author: Valentino <[email protected]>
Date:   Thu Oct 10 08:55:28 2013 -0400

    Merge remote-tracking branch 'origin/master' into vs-github-enterprise

commit aec25db
Author: Valentino <[email protected]>
Date:   Thu Oct 10 08:53:35 2013 -0400

    [fix] Github endpoint and site urls, review command using custom title and message flags, and upgrade gems

commit 7b7b706
Author: Valentino <[email protected]>
Date:   Wed Oct 9 14:26:13 2013 -0400

    Display git commands as they run, and add flags for review title and message

commit 2d4cda5
Author: Valentino <[email protected]>
Date:   Tue Oct 1 14:34:23 2013 -0400

    Enable use of Github Enterprise accounts
@codenamev
Copy link
Collaborator

@markkevans I've updated the gem with a few new setup switches that should allow you to configure GitReflow to use your Github Enterprise account. Please review the readme and let me know if it works for you as I have no way of testing this myself. I'll close once I get an 👌 from you.

@markkevans
Copy link
Author

Great thanks! I'll test and confirm in the next couple of days

@nhance
Copy link
Member

nhance commented Nov 14, 2013

Hey @markkevans, any luck with this? I'd love to close this ticket out /cc @codenamev

@markkevans
Copy link
Author

hi @nhance sorry for the delay.

I've tested this and the config part seems to work, I do get an error when using reflow review and reflow status which just says

error: undefined method `[]' for nil:NilClass

Is there a way I can debug where that is being thrown?

@codenamev
Copy link
Collaborator

@markkevans Can you try upgrading and see if this still happens? We fixed that issue in a recent release.

@zmbush
Copy link

zmbush commented Apr 10, 2014

I just ran into this problem as well.

The exception came from the following code:

def remote_user
  gh_remote_user = `git config --get remote.origin.url`.strip
  gh_remote_user.slice!(/github\.com[\/:](\w|-|\.)+/i)[11..-1]
end

Which i fixed by changing it to:

def remote_user
  gh_remote_user = `git config --get remote.origin.url`.strip
  gh_remote_user.slice!(/[\/:](\w|-|\.)+/i)[1..-1]
end

This function assumes that the remote url will be for github.com

@codenamev
Copy link
Collaborator

@zmbush we've made a pretty significant overhaul and have fixed those issues in 0.4.1

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

No branches or pull requests

4 participants