Skip to content

Commit a5b9f6a

Browse files
committed
disabling pylint for management packages. generated could shouldn't be scanned, just a waste of processing time right now
1 parent e43833e commit a5b9f6a

File tree

2 files changed

+13
-10
lines changed

2 files changed

+13
-10
lines changed

eng/pipelines/templates/jobs/archetype-sdk-nightly.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -124,6 +124,8 @@ jobs:
124124
dependsOn:
125125
- 'Build'
126126

127+
timeoutInMinutes: 180
128+
127129
pool:
128130
vmImage: $(OSVmImage)
129131

eng/tox/run_pylint.py

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -40,16 +40,17 @@
4040
package_name = os.path.basename(args.target_package)
4141

4242
try:
43-
check_call(
44-
[
45-
sys.executable,
46-
"-m",
47-
"pylint",
48-
"--rcfile={}".format(rcFileLocation),
49-
"--output-format=parseable",
50-
os.path.join(args.target_package, "azure"),
51-
]
52-
)
43+
if "mgmt" not in package_name:
44+
check_call(
45+
[
46+
sys.executable,
47+
"-m",
48+
"pylint",
49+
"--rcfile={}".format(rcFileLocation),
50+
"--output-format=parseable",
51+
os.path.join(args.target_package, "azure"),
52+
]
53+
)
5354
except CalledProcessError as e:
5455
logging.error(
5556
"{} exited with linting error {}".format(package_name, e.returncode)

0 commit comments

Comments
 (0)