Skip to content

Conversation

@mockdeep
Copy link
Contributor

@mockdeep mockdeep commented Feb 7, 2021

No description provided.

@mockdeep mockdeep assigned mockdeep and unassigned mockdeep Feb 7, 2021
@mockdeep mockdeep marked this pull request as draft February 7, 2021 19:05
@mockdeep mockdeep force-pushed the rf-no_docker_ci_ga branch 22 times, most recently from 2ba7d5a to a795a4d Compare February 7, 2021 20:52
@managedkaos
Copy link
Collaborator

@mockdeep if you pull this one out of draft, I'm ready to move forward with it.

@mockdeep mockdeep marked this pull request as ready for review February 14, 2021 20:24
Copy link
Collaborator

@managedkaos managedkaos left a comment

Choose a reason for hiding this comment

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

@mockdeep Any reason why the codeclimate workflow was removed? Just wondering...

@mockdeep
Copy link
Contributor Author

@managedkaos codeclimate is already reporting from its own infrastructure, so it seemed redundant to also be running it on Github Actions.

code-climate

Copy link
Contributor

@rlgreen91 rlgreen91 left a comment

Choose a reason for hiding this comment

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

LGTM, @managedkaos just let me know when you approve it and I'll merge.

Copy link
Collaborator

@managedkaos managedkaos left a comment

Choose a reason for hiding this comment

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

LGTMe!

@rlgreen91 rlgreen91 merged commit 484f3ac into main Feb 24, 2021
@rlgreen91 rlgreen91 deleted the rf-no_docker_ci_ga branch February 24, 2021 03:53
@managedkaos managedkaos linked an issue Apr 24, 2021 that may be closed by this pull request
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.

Run Rails in the GitHub Actions Container for CI

4 participants