Skip to content
This repository has been archived by the owner on Mar 27, 2024. It is now read-only.

Code styling #103

Open
cdjackson opened this issue Aug 29, 2016 · 2 comments
Open

Code styling #103

cdjackson opened this issue Aug 29, 2016 · 2 comments
Labels

Comments

@cdjackson
Copy link
Collaborator

It seems you enabled a CI tonight for code formatting and it has pretty tight requirements on how the code is styled. I've spent a lot of time tonight trying to get this correct as I've had well over 100 emails from the CI bot. Please can you either provide information on how you want it styled or disable this. It could easily spend days guessing what this thing wants.

I tried googling for a formatter file (I'm using Eclipse) but couldn't find anything. I can't believe there's not something available given how picky the bot is. I've tried using a formatter for another project I work on that has tight requirements, and that made things worse - sorry!

I think the only way this will work is if there is an auto formatter somewhere - either in the editor, or as part of the CI (which probably doesn't work as it would need to commit to git).

@cdjackson
Copy link
Collaborator Author

@tlaukkan any chance of disabling the code review please. Otherwise I'll close all PRs until this is sorted? When I first created these PRs over the weekend the spam generator didn't do anything, so I don't know if you enabled it earlier in the week?

@tlaukkan
Copy link
Owner

tlaukkan commented Dec 4, 2016

Hi. I am sorry about the inconvenience. I think the style checks had been on for a while. I do not know why it suddenly activated for you. In general I think it is useful to have it when changes are smaller but on larger refactoring it does spam quite much. The code style requirements are google Java code style conventions and you are right, we should include definition file to the project.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants