The Wayback Machine - https://web.archive.org/web/20201020142848/https://github.com/codeclimate/codeclimate-rubocop/pull/161
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

Migrate to circle2 #161

Merged
merged 1 commit into from Mar 21, 2019
Merged

Migrate to circle2 #161

merged 1 commit into from Mar 21, 2019

Conversation

@ale7714
Copy link
Member

@ale7714 ale7714 commented Mar 21, 2019

Old branches will need to cherry pick this commit

@ale7714 ale7714 requested a review from codeclimate-hermes Mar 21, 2019
@codeclimate-hermes codeclimate-hermes requested review from maxjacobson and removed request for codeclimate-hermes Mar 21, 2019
Copy link
Collaborator

@filipesperandio filipesperandio left a comment

I think circle.yml removal should be in this PR too. No?

steps:
- checkout
- run: make image
- run: make citest

This comment has been minimized.

@maxjacobson

maxjacobson Mar 21, 2019
Contributor

per the failing build, it looks like this should be make test

@ale7714
Copy link
Member Author

@ale7714 ale7714 commented Mar 21, 2019

Sorry this is WIP, requested review too eagerly. Wrapping up one more thing. And yes to both things

Old branches will need to cherry pick this commit
@ale7714 ale7714 force-pushed the ap/circle2 branch from ee4c02b to 1c85520 Mar 21, 2019
steps:
- checkout
- run:
name: Validate owner

This comment has been minimized.

@ale7714

ale7714 Mar 21, 2019
Author Member

circle 1 has this nice feature https://github.com/codeclimate/codeclimate-rubocop/blob/master/circle.yml#L24.

Unfortunately not available in circle2 so we're the validation within the job itself.

circleci step halt is a nifty trick to exit the job without failing the build. I learn it from this post.

This comment has been minimized.

@maxjacobson

maxjacobson Mar 21, 2019
Contributor

That is neat!

@ale7714 ale7714 requested review from filipesperandio and maxjacobson Mar 21, 2019
@ale7714 ale7714 merged commit 85f4bd0 into master Mar 21, 2019
2 checks passed
2 checks passed
build_deploy Workflow: build_deploy
Details
codeclimate All good!
Details
@ale7714 ale7714 deleted the ap/circle2 branch Mar 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
You can’t perform that action at this time.