Skip to content

Added CheckStyle rule enforcing indentation guides - #297

Open
brownian-motion wants to merge 1 commit into
hamcrest:masterfrom
brownian-motion:issue295_codeStyle
Open

brownian-motion wants to merge 1 commit into
hamcrest:masterfrom
brownian-motion:issue295_codeStyle

Conversation

@brownian-motion

Copy link
Copy Markdown

Discussion in issues like #295 suggest a desire for a consistent code style. This PR seeks to implement that.

Enforced indentation style of 2 spaces (with further details matching Google's spec).
Formatted existing .java code to match.

@nhojpatrick

nhojpatrick commented Jun 29, 2020

Copy link
Copy Markdown
Member

@brownian-motion please can you rebase from master, as hamcrest-core and hamcrest-library have been refactored a lot and also deprecated, so that everything is just in hamcrest.

@nhojpatrick

Copy link
Copy Markdown
Member

Going to try and kick start hamcrest, so if you want to get it merged, please rebase from the branch v2.3-candidates.
Still trying to understand how has permissions to perform a release.

@nhojpatrick
nhojpatrick force-pushed the master branch 2 times, most recently from 9bc653b to e9f7fc8 Compare February 13, 2022 17:06

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: Needs triage

Development

Successfully merging this pull request may close these issues.

2 participants