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

Add github action for running tests #1624

Merged
merged 1 commit into from
Jul 16, 2020
Merged

Add github action for running tests #1624

merged 1 commit into from
Jul 16, 2020

Conversation

sethboyles
Copy link
Member

Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:

  • A short explanation of the proposed change:

  • An explanation of the use cases your change solves

  • Links to any other associated PRs

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the master branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/173024538

The labels on this github issue will be updated when the story is started.

@matt-royal
Copy link
Contributor

This would make us much more effective at reviewing PRs, so I'm taking a stab at getting this working.

Co-authored-by: Seth Boyles <sboyles@pivotal.io>
Co-authored-by: Jaskanwal Pawar <jpawar@pivotal.io>
Co-authored-by: Matt Royal <mroyal@pivotal.io>
@matt-royal
Copy link
Contributor

matt-royal commented Jul 16, 2020

I was able to get the work @sethboyles and @jspawar started to pass. The main issue was that we were using localhost when opening a TCPSocket. This works on Mac, but on linux only 127.0.0.1 works. I also added the bundle install step for rubocop. Otherwise everything is as originally implemented.

@matt-royal matt-royal merged commit 725654e into master Jul 16, 2020
@tjvman tjvman deleted the github-actions branch August 24, 2021 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants