Skip to content

changed coala-analyzer to coala-eclipse#43

Open
yukgupta wants to merge 1 commit into
coala:masterfrom
yukgupta:master
Open

changed coala-analyzer to coala-eclipse#43
yukgupta wants to merge 1 commit into
coala:masterfrom
yukgupta:master

Conversation

@yukgupta

Copy link
Copy Markdown

fixes #33

@yukgupta

Copy link
Copy Markdown
Author

@yukiisbored My CI test is failing, where did I go wrong?

@mamat-rahmat

mamat-rahmat commented Nov 24, 2017

Copy link
Copy Markdown
Member

Looks like an intermittent error. Nothing is wrong with your change.

@mamat-rahmat mamat-rahmat left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please add tag with file that is modified, in front of your commit messsage.
And better rephrase to explain that it is changing build status url.

@mamat-rahmat

Copy link
Copy Markdown
Member

unack 8f88d56

@yukiisbored

Copy link
Copy Markdown
Member

@xyzyg Hello, it looks like a test failed. I'm gonna restart CI and please change your commit message by amending it. Refer to coala.io/commit for more information.

@yukiisbored

yukiisbored commented Nov 24, 2017

Copy link
Copy Markdown
Member

Actually, I think one of the tests is broken.

@yukiisbored

yukiisbored commented Nov 24, 2017

Copy link
Copy Markdown
Member

Gonna mark this as blocked. The same way #38 is handled.

@jayvdb jayvdb changed the title changed coala-analyzer to coala changed coala-analyzer to coala-eclipse Nov 27, 2017
@jayvdb

jayvdb commented Nov 27, 2017

Copy link
Copy Markdown
Member

The commit message is wrong shortlog and wrong Fixes.

@Monal5031

Copy link
Copy Markdown
Member

@xyzyg Please assign yourself to the issue before working on it.

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

Development

Successfully merging this pull request may close these issues.

Change README Travis Badge link

6 participants