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

Trafico seems not working #3

Open
nhuray opened this issue Mar 10, 2021 · 7 comments
Open

Trafico seems not working #3

nhuray opened this issue Mar 10, 2021 · 7 comments

Comments

@nhuray
Copy link

nhuray commented Mar 10, 2021

Summary

I installed Trafico bot and add the file .github/trafico.yml in the main branch (default)

When I open a pull request, there's no labels set by Trafico bot.

Steps to reproduce

  • Install Trafico for your user or organization and define repositories
  • Add a file .github/trafico.yml following the documentation on the default branch
  • Open a pull request

What is the current bug behavior?

The pull request is not labeled by Trafico bot.

What is the expected correct behavior?

The Pull request should be labeled with Needs Review label.

Screenshots

Org or User Settings

Screen Shot 2021-03-09 at 8 24 17 PM

Trafico config

StaticShot_09-03-2021_20-25-15

Pull request

StaticShot_09-03-2021_20-26-08

@shahzebsiddiqui
Copy link

I echo @nhuray i have trafico configured for https://github.com/buildtesters/buildtest and it was working in the past some time back for instance see this PR: buildtesters/buildtest#225 it has the Merged label. Now if you see my latest PRs https://github.com/buildtesters/buildtest/pulls i dont see any labels as i move PRs from state. I have a trafico configuration file https://github.com/buildtesters/buildtest/blob/devel/.github/trafico.yml which is based of sample file. I take it this is not required only if we need to configure it.

I tried reinstalling the app several times and reopen a PR a few times and it didnt work. You can look at buildtesters/buildtest#687 and see the behavior.

@nhuray
Copy link
Author

nhuray commented Mar 15, 2021

@orhan Are you still maintaining trafico ?

@orhan
Copy link
Owner

orhan commented Mar 15, 2021

@nhuray
To be honest I haven't worked on this project in a long time and it was never finished, as I couldn't even convince my team back then to actually use it.

But I'll actually try to look into this issue sometime during the week!

@shahzebsiddiqui
Copy link

Thanks @orhan this looks pretty cool, it would be nice to get this up and running.

@rlataguerra
Copy link

hi @nhuray our team at @wecraftapps use it and love it, thanks for your work ❤️

@nhuray
Copy link
Author

nhuray commented Mar 30, 2021

Hi @rlataguerra, thanks for the support but actually I'm not the author of this project. @orhan is the author !

Are you saying you setup Trafico for your team and the app is working for you ?

@rlataguerra
Copy link

rlataguerra commented Mar 30, 2021

@nhuray nope, not working anymore.

Sorry for the ping, I meant to ping @orhan ❤️

shahzebsiddiqui added a commit to buildtesters/buildtest that referenced this issue Apr 13, 2021
shahzebsiddiqui added a commit to buildtesters/buildtest that referenced this issue Apr 13, 2021
* remove trafico workflow because its not working see orhan/trafico#3

* remove unused import
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

No branches or pull requests

4 participants