-
-
Notifications
You must be signed in to change notification settings - Fork 27
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
Issue #6 | Add SqlServer support | Adarsh #22
Conversation
@italolelis I see an error in coverall |
180102a
to
db17886
Compare
Hey @m4ver1k, Thank you for contribution. Regarding the failing CI, I think it has something to do with the way GithubActions work in a forked PR. The env var is set in this repo, but if the PR comes from a fork, it doesn't have access to that variable. I'll see if I can only run that step in master or something like that. |
@m4ver1k All right, now I have split the workflow for master and pull requests. I also updated our linting version, and now the PR has a few linting errors. If you don't mind fixing them I would appreciate it. Thanks!! |
Sure will do that |
@italolelis I have fixed the linting issues. |
@m4ver1k Thank you very much for your contribution! 👍 |
No description provided.