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

Split one instance variable per line #182

Merged
merged 1 commit into from Oct 4, 2014

Conversation

2 participants
@cirosantilli
Copy link
Contributor

commented Oct 4, 2014

It's clearer. I spent 5 secs trying to understand what was the relation between actor and repo path.

Razer6 added a commit that referenced this pull request Oct 4, 2014

Merge pull request #182 from cirosantilli/split-instance-var-defs
Split one instance variable per line

@Razer6 Razer6 merged commit c35ba96 into gitlabhq:master Oct 4, 2014

1 check passed

default The build passed on Semaphore.
Details

@cirosantilli cirosantilli deleted the cirosantilli:split-instance-var-defs branch Oct 4, 2014

maxlazio pushed a commit that referenced this pull request Apr 4, 2019

Merge branch '182-lib-gitlab_net-rb-37-in-check_access-uninitialized-…
…constant-gitlabnet-gitaccessstatus-nameerror' into 'master'

Explicitly include gitlab_access_status.rb

Closes #182

See merge request gitlab-org/gitlab-shell!292
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.