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

Use codeclimate-action to improve edge-case handling #73

Merged
merged 1 commit into from
Aug 9, 2020

Conversation

jcypret
Copy link
Owner

@jcypret jcypret commented Aug 9, 2020

Code Climate does not have an official GitHub action, but there are a
few tricky cases that need worked around making it undesirable to use
the shell scripts directly. What has caused the most headaches is the
absence of required ENV variables for branch and SHA that need
conditional handling depending on where the commit came from (commit,
pull request, fork, etc). By using a community plugin, the hope is to
simplify action config for this project and avoid these gotchas.

This also pins the version of SimpleCov to 0.17.x to fix an issue where
Code Climate's reporter does not current support versions >= 0.18
(paambaati/codeclimate-action#102 (comment)).

Code Climate does not have an official GitHub action, but there are a
few tricky cases that need worked around making it undesirable to use
the shell scripts directly. What has caused the most headaches is the
absence of required ENV variables for branch and SHA that need
conditional handling depending on where the commit came from (commit,
pull request, fork, etc). By using a community plugin, the hope is to
simplify action config for this project and avoid these gotchas.

This also pins the version of SimpleCov to 0.17.x to fix an issue where
Code Climate's reporter does not current support versions >= 0.18
(paambaati/codeclimate-action#102 (comment)).
@jcypret jcypret merged commit a004c05 into master Aug 9, 2020
@jcypret jcypret deleted the codeclimate-action branch August 9, 2020 05:08
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.

1 participant