-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[CI] Move 1: Copy job SourceStaticPy from Travis to ADO (5/6) #1156
[CI] Move 1: Copy job SourceStaticPy from Travis to ADO (5/6) #1156
Conversation
If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
|
I'm curious why the PR title is the same as the one you created last time? In reply to: 566479337 [](ancestors = 566479337) |
Not the same. The old one is [CI] Move 1: Copy job SourceStaticPy from Travis to ADO (1/6) |
So how many sub-steps are left for your move 1? In reply to: 566863429 [](ancestors = 566863429) |
None. This is the last step of Move 1. |
…1156) * Enable flake8 on extension in ADO * Enable pylint on CI scripts in ADO * Enable flake8 on CI scripts in ADO * Enable code owner and license checks in ADO * Fix bug: pylint==1.9.5 didnt recognize disable=too-many-lines
This is the first move for CI that migrate from Travis to ADO which will move job SourceStaticPy to ADO and renamed it as StaticAnalysis.
This PR is 5/6 part of this move. Basically done is to enable the rest of checking in ADO:
Respect #867
This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines
azdev style <YOUR_EXT>
locally? (pip install azdev
required)python scripts/ci/test_index.py -q
locally?For new extensions: