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

Fix current Rubocop Issues #111

Merged
merged 15 commits into from
Apr 22, 2022
Merged

Fix current Rubocop Issues #111

merged 15 commits into from
Apr 22, 2022

Conversation

mortik
Copy link
Contributor

@mortik mortik commented Feb 17, 2022

Should also fix the following Stories:

SC-10366

@shortcut-integration
Copy link

@mortik mortik force-pushed the chore/rubocop branch 2 times, most recently from 9f99ba8 to 7086c55 Compare February 18, 2022 08:37
@ChristianBreitkreutz
Copy link

LooksGoodToMe

@ChristianBreitkreutz ChristianBreitkreutz removed their assignment Feb 22, 2022
Copy link

@SteffenHam SteffenHam left a comment

Choose a reason for hiding this comment

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

I like having Rubocop as an extra step in the CI 👍

name: Install ruby dependencies
command: gem update --system; bin/setup
- run:
name: Execute specs

Choose a reason for hiding this comment

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

How about a more descriptive name? Something like Run Rubocop maybe?

.circleci/config.yml Outdated Show resolved Hide resolved
@mortik mortik removed their assignment Apr 21, 2022
@codeclimate
Copy link

codeclimate bot commented Apr 22, 2022

Code Climate has analyzed commit 800c758 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 76.9% (50% is the threshold).

This pull request will bring the total coverage in the repository to 90.1% (0.5% change).

View more on Code Climate.

Copy link

@SteffenHam SteffenHam left a comment

Choose a reason for hiding this comment

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

RTM 👍

@mortik mortik merged commit 571678d into master Apr 22, 2022
@mortik mortik deleted the chore/rubocop branch April 22, 2022 09:16
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.

3 participants