-
-
Notifications
You must be signed in to change notification settings - Fork 70
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 CI issue #231
Fix CI issue #231
Conversation
Hey @pouriyajamshidi I was expecting Codefactor to throw the same CI error I faced in #225 did something change? |
Hey @SYSHIL No, nothing has changed code-base wise. Although, I am not sure if empty commits cause the One surprising thing is that the CI is running without approval. In actual PRs, I have to always click |
yep, I can confirm codeQL triggers on code changes! Best part is it does only check the parts you change and not the overall code-base :) |
Heyy @pouriyajamshidi sorry I missed this message from you, had a busy month...thanks for the knowledge share! |
This pull request has been automatically marked as stale because it has not had recent activity. Thank you for your contribution! |
This pull request has been automatically closed because it has not had recent activity or follow ups. |
No description provided.