Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix gitmate's complain #20

Closed
wants to merge 1 commit into from
Closed

Fix gitmate's complain #20

wants to merge 1 commit into from

Conversation

9r0x
Copy link

@9r0x 9r0x commented Nov 7, 2017

This fixes broken ci, and avoids gitmate's complain

I think .coafile is only needed for functioning code in this repo, so I excluded python's config and injected it manually each time ci is running.

Checklist

  • [√] I read the commit guidelines and I've followed
    them.
  • [√] I ran coala over my code locally. (All commits have to pass
    individually.
    It is not sufficient to have "fixup commits" on your PR,
    our bot will still report the issues for the previous commit.) You will
    likely receive a lot of bot comments and build failures if coala does not
    pass on every single commit!

This fixed broken ci, and avoids gitmate's complain
@jayvdb
Copy link
Member

jayvdb commented Nov 7, 2017

What is the problem that you are trying to solve ?
Which issue ?

@9r0x
Copy link
Author

9r0x commented Nov 7, 2017

There is a problem with gitmate as it is checking the test python file, see pr #18
So I removed python setting in .coafile

@9r0x
Copy link
Author

9r0x commented Nov 7, 2017

Also the badge in readme is showing error, I guess my pr will solve it.
After this pr, later I will try to add coala checker to default emacs checker

@9r0x 9r0x mentioned this pull request Nov 7, 2017
@jayvdb
Copy link
Member

jayvdb commented Nov 7, 2017

#18 is red because a maintainer flagged it for needing improvement, not a build failure.

https://travis-ci.org/coala/coala-emacs/builds/255744435 did fail. That was #17

Copy link
Member

@jayvdb jayvdb left a comment

Choose a reason for hiding this comment

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

This is not the solution. This is breaking our CI more.
Please finish your other PRs.

@jayvdb
Copy link
Member

jayvdb commented Nov 7, 2017

No, this is a difficult problem, and you need to finish the newcomer process.

@jayvdb jayvdb closed this Nov 7, 2017
@9r0x 9r0x changed the title Fixes Broken CI Fix gitmate's complain Nov 9, 2017
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.

3 participants