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

Enable travis-buddy on GitHub repos #114

Open
jayvdb opened this issue Apr 11, 2018 · 23 comments
Open

Enable travis-buddy on GitHub repos #114

jayvdb opened this issue Apr 11, 2018 · 23 comments
Assignees

Comments

@jayvdb
Copy link
Member

jayvdb commented Apr 11, 2018

https://github.com/bluzi/travis-buddy provides a synopsis of the travis error on the pull request

https://travisbuddy.com

@jayvdb
Copy link
Member Author

jayvdb commented Apr 11, 2018

@manankalra
Copy link

There should be individual issues opened for this in the corresponding repos or just this issue should be referenced in the individual PRs?

@jayvdb
Copy link
Member Author

jayvdb commented Apr 12, 2018

No need, just use Related to https://github.com/coala/meta/issues/114

@palash25
Copy link
Member

Can this issue have multiple assignee? I think we can complete it much faster if we divide individual repos amongst ourselves. If you don't mind @manankalra .

cc @jayvdb

@jayvdb
Copy link
Member Author

jayvdb commented Apr 12, 2018

Please read the issue again @palash25 .

@jayvdb
Copy link
Member Author

jayvdb commented Apr 22, 2018

Another option is to enable travieso

maybe we try that after travis-buddy, if we dont like travis-buddy.

c.f. coala/coala#4891

manankalra added a commit to manankalra/documentation that referenced this issue Apr 22, 2018
manankalra added a commit to manankalra/documentation that referenced this issue Apr 27, 2018
@manankalra
Copy link

travis isn't set up for cEPs; needed?

@jayvdb
Copy link
Member Author

jayvdb commented May 2, 2018

coala/documentation#555 (comment) - a buddy error. looks nice!

travis isn't set up for cEPs; needed?

ok, skip that one for now.

@manankalra
Copy link

manankalra commented May 2, 2018

For corobo:
As per my knowledge, travis-ci/travis-ci#5121 isn't solved yet. Also, the docs do not mention anything of this sort - https://docs.travis-ci.com/user/notifications/#Configuring-webhook-notifications.

Will have to omit on_success: never for now.

manankalra added a commit to manankalra/corobo that referenced this issue May 2, 2018
manankalra added a commit to manankalra/coala-vs-code that referenced this issue May 2, 2018
manankalra added a commit to manankalra/projects that referenced this issue May 2, 2018
manankalra added a commit to manankalra/coala-html that referenced this issue May 2, 2018
manankalra added a commit to manankalra/community that referenced this issue May 2, 2018
@jayvdb
Copy link
Member Author

jayvdb commented May 3, 2018

manankalra added a commit to manankalra/gh-board that referenced this issue May 3, 2018
manankalra added a commit to manankalra/coala-quickstart that referenced this issue May 3, 2018
manankalra added a commit to manankalra/coala-emacs that referenced this issue May 3, 2018
manankalra added a commit to manankalra/gci-leaders that referenced this issue May 3, 2018
manankalra added a commit to manankalra/landing-frontend that referenced this issue May 3, 2018
@manankalra
Copy link

Should I open PRs for all of them?
There seems to be no ongoing contribution activities in any of the above repos (other than gh-board). Shouldn't we first test it on coala-bears then?

@bluzi
Copy link

bluzi commented May 26, 2018

Hey guys,

It looks like you've been using TravisBuddy for more than a month now, so I was just wondering if there's anything you think we can improve or add?

@jayvdb
Copy link
Member Author

jayvdb commented Jun 7, 2018

Interesting missing error at coala/corobo#550 (comment)

@manankalra
Copy link

Here, too. - coala/corobo#552

@li-boxuan
Copy link
Member

TravisBuddy copies too much log. 😕 That's a lot of network traffic, especially when we use API to fetch comments.

@jayvdb
Copy link
Member Author

jayvdb commented Jun 9, 2018

Probably bluzi/travis-buddy#109 @bluzi

Also some feature requests https://github.com/bluzi/travis-buddy/issues/created_by/jayvdb

@li-boxuan , can you link to some large comments. @bluzi may know how to fix them.

@li-boxuan
Copy link
Member

This is a large comment: coala/community#143 (comment)

@jayvdb
Copy link
Member Author

jayvdb commented Jun 9, 2018

Any large comments in JS projects?

TravisBuddy doesnt have code for python specific projects yet.

@bluzi
Copy link

bluzi commented Jun 9, 2018

Thanks a lot guys! that's super helpful!


Interesting missing error at coala/corobo#550 (comment)

Here, too. - coala/corobo#552

Thanks, will definitely look into it


Also some feature requests https://github.com/bluzi/travis-buddy/issues/created_by/jayvdb

Awesome, thank you!


This is a large comment: coala/community#143 (comment)

Definitely a bug, TravisBuddy should break the log using the scripts supplied in .travis.yml. Instead this comment contains the entire raw log.
Will look into it and get back to you.

@li-boxuan
Copy link
Member

Not sure if this is also a 'large' comment (JS project) coala/gci-leaders#149 (comment)

@bluzi
Copy link

bluzi commented Jun 9, 2018

Not sure if this is also a 'large' comment (JS project) coala/gci-leaders#149 (comment)

Nope.
As you can see in the build log, the raw log is a lot bigger. TravisBuddy extracted the log of the executed command (npm run build), so that's by design.

gitmate-bot pushed a commit to manankalra/gci-leaders that referenced this issue Sep 10, 2018
@jayvdb
Copy link
Member Author

jayvdb commented Oct 31, 2018

@manankalra , any active repos left to enable this in?

@manankalra
Copy link

Yes. coala and coala-bears.

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

No branches or pull requests

5 participants