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

bumb gitlab API version #1684

Merged
merged 1 commit into from
Aug 29, 2017
Merged

Conversation

elliott-beach
Copy link
Contributor

@elliott-beach elliott-beach commented Aug 27, 2017

Fixes #1658.

I'm not sure what level of testing is needed - the ruby tests pass, but as for verifying if the site actually works I'm not sure what to do.

Two questions:

  1. Where is the GitLab API exercised in the site? I imagine this is would be GitLab repositories that are indexed, but it's good to know what the changes could potentially impact.
  2. How to populate my local version of the site with data that exercises the GitLab API, once I know where it is?

@elliott-beach
Copy link
Contributor Author

OK, an example of a gitlab-hosted repo is https://libraries.io/rubygems/gitaly. I'll try and get that ruby gem.

@elliott-beach
Copy link
Contributor Author

elliott-beach commented Aug 27, 2017

I tested on that repo and navigating to the gitlab url works, so I this may be good to go.

@andrew andrew merged commit 055aad5 into librariesio:master Aug 29, 2017
@andrew
Copy link
Contributor

andrew commented Aug 29, 2017

Looks good, thanks @e-beach!

andrew added a commit that referenced this pull request Sep 12, 2017
* master: (26 commits)
  Add concurrent-ruby-ext gem to speed up sidekiq
  Update appsignal
  Updated capistrano version lock
  Updated rails to 5.0.6
  Update dependencies
  Update README.md
  Update docs to reflect that watcher is now dispatch
  Fix typo
  Update pictogram with latest puppet logo
  Update counter_culture, fixes #1690
  Update dependencies
  Show all licenses in some places, fixes #1678
  Improve docs around running a background worker
  Handle download_members being called on github users gracefully
  bumb gitlab API version (#1684)
  Update dependencies
  Don't strip booleans
  Update brakeman
  Support markdown readmes for gitlab and bitbucket
  Fix double escaping issues with gitlab api
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants