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
Circular dependency in v3.0.0 prevents webhook from working #1044
Comments
A little more investigation shows that the issue is that src/server/src/services/webhook.js requires repo.js which in turn requires webhook.js. repo.js also requires org.js which in turn also requires webhook.js. |
gtjoseph
added a commit
to asterisk/cla-assistant
that referenced
this issue
Feb 22, 2024
There was a circular dependency between repo, org and webhook that prevented webhook calls from GitHub from being serviced. Merging those files together resolves the circular dependency. Resolves: cla-assistant#1044
gtjoseph
added a commit
to asterisk/cla-assistant
that referenced
this issue
Feb 23, 2024
There was a circular dependency between repo, org and webhook that prevented webhook calls from GitHub from being serviced. Merging those files together resolves the circular dependency. Resolves: cla-assistant#1044
gtjoseph
added a commit
to asterisk/cla-assistant
that referenced
this issue
Feb 23, 2024
There was a circular dependency between repo, org and webhook that prevented webhook calls from GitHub from being serviced. Merging those files together resolves the circular dependency. Resolves: cla-assistant#1044
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
URL to the linked Repository or Organization
https://github.com/
Steps to reproduce the problem
What is the expected result?
The check shows status of CLA signature
What happens instead?
The check shows:
license/cla Expected — Waiting for status to be reported
permanently
Any other information? (attach screenshot if possible)
The server shows the following error:
Used browser and browser version?
Any other tested browsers/devices(OK/FAIL):
The text was updated successfully, but these errors were encountered: