Skip to content

coala: Add coala support#67

Merged
gaocegege merged 2 commits intomasterfrom
code-style/coala
May 19, 2017
Merged

coala: Add coala support#67
gaocegege merged 2 commits intomasterfrom
code-style/coala

Conversation

@gaocegege
Copy link
Copy Markdown
Member

Closes #52
Ref coala/coala-bears#1750

Signed-off-by: Ce Gao ce.gao@outlook.com

Ref coala/coala-bears#1750

Signed-off-by: Ce Gao <ce.gao@outlook.com>
@gaocegege
Copy link
Copy Markdown
Member Author

coala is a perfect tool to lint and format the code in command line. But it seems that there is something wrong. I will comment the checkstyle bear now and uncomment it until the bug is fixed.

Signed-off-by: Ce Gao <ce.gao@outlook.com>
@gaocegege gaocegege merged commit c64a66f into master May 19, 2017
@gaocegege gaocegege deleted the code-style/coala branch May 19, 2017 12:59
@jeremydouglass
Copy link
Copy Markdown
Member

Excellent! Good use of coala to clean up the example sketches.

Please create an Issue for the checkstyle bear issue that references the appropriate commit line.

Please clarify what "add coala support" means -- give a bit more detail here and in the log file. For example, it could mean support for running coala manually against the repo, or support for Travis-ci integration of coala, or support for invoking coala from PDE. This will help non-core developers understand how they could use coala with Processing.R

@gaocegege
Copy link
Copy Markdown
Member Author

#67 has been filed :) And I have added the introduction in #68 , I think it is better to add the detail into the documentation 🤔

WDYT

@jeremydouglass
Copy link
Copy Markdown
Member

jeremydouglass commented May 19, 2017

Good -- yes, I agree with you that the Coding Style section is the best place for it.

Good inclusion of references, e.g. to FormatR. Is the coala add.r module (bear) based on FormatR, and/or is it designed to follow the same rules?

@gaocegege
Copy link
Copy Markdown
Member Author

Yes, the FormatRBear uses formatR as the backend.

FormatRBear

Check and correct formatting of R Code using known formatR utility.

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.

2 participants