Skip to content
This repository has been archived by the owner on Sep 9, 2022. It is now read-only.

Introduce RuboCop & SideCI #242

Merged
merged 4 commits into from
Jul 7, 2016
Merged

Introduce RuboCop & SideCI #242

merged 4 commits into from
Jul 7, 2016

Conversation

dtan4
Copy link
Owner

@dtan4 dtan4 commented Jul 6, 2016

WHY

Now a lot of developers contributed to Terraforming. However it seems that there lack of consistency of coding style.
I'd like to standalize the coding style in this repository.

WHAT

Add RuboCop configratiton file to declare the coding style.
I also introduce SideCI to check the style of pushed patch automatically.

@coveralls
Copy link

coveralls commented Jul 6, 2016

Coverage Status

Coverage remained the same at 97.335% when pulling 7637c6c on rubocop into 967bcb7 on master.

@coveralls
Copy link

coveralls commented Jul 6, 2016

Coverage Status

Coverage remained the same at 97.335% when pulling 067652b on rubocop into 967bcb7 on master.

@coveralls
Copy link

coveralls commented Jul 7, 2016

Coverage Status

Coverage remained the same at 97.335% when pulling 8fad911 on rubocop into 967bcb7 on master.

@dtan4 dtan4 merged commit cd0a7d2 into master Jul 7, 2016
@dtan4 dtan4 deleted the rubocop branch July 7, 2016 01:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants