Skip to content

Commit 63874c2

Browse files
committed
CNJR-6431: Fix/ignore Pylint issues
1 parent 6dcbb75 commit 63874c2

File tree

7 files changed

+13
-9
lines changed

7 files changed

+13
-9
lines changed

.github/CODEOWNERS

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
# org the new repo will be in. GitHub automatically applies the correct version.
33

44
# Uncomment the appropriate team line to automatically tag the owning team on PRs
5-
@cyberark/community-and-integrations-team @conjurinc/community-and-integrations-team @conjurdemos/community-and-integrations-team @conjur-enterprise/community-and-integrations
5+
* @cyberark/community-and-integrations-team @conjurinc/community-and-integrations-team @conjurdemos/community-and-integrations-team @conjur-enterprise/community-and-integrations
66

77
# Changes to .trivyignore require Security Architect approval
88
.trivyignore @cyberark/security-architects @conjurinc/security-architects @conjurdemos/security-architects @conjur-enterprise/conjur-security

ci/publish/Dockerfile.publish

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
FROM ubuntu:23.04
22
ENV INSTALL_DIR=/opt/conjur-api-python3
3-
ENV DEBIAN_FRONTEND noninteractive
3+
ENV DEBIAN_FRONTEND=noninteractive
44
RUN apt-get update && \
55
apt-get install -y bash \
66
binutils \

ci/test/.pylintrc

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,6 @@
1+
[MESSAGES CONTROL]
2+
# Disable "too-many-positional-arguments".
3+
# Can't be ignored individually b/c it's not a known rule for the pylint hook running in CI.
4+
disable=R0917
15
[FORMAT]
2-
max-line-length=120
6+
max-line-length=120

ci/test/Dockerfile.test

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
FROM ubuntu:23.04
22
ENV INSTALL_DIR=/opt/conjur-api-python
3-
ENV DEBIAN_FRONTEND noninteractive
3+
ENV DEBIAN_FRONTEND=noninteractive
44

55
RUN apt-get update && \
66
apt-get install -y bash \

conjur_api/client.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
# Builtins
1111
import json
1212
import logging
13-
from typing import Optional
13+
from typing import Optional, Tuple
1414

1515
from conjur_api.errors.errors import ResourceNotFoundException, MissingRequiredParameterException, HttpStatusError
1616
from conjur_api.http.api import Api
@@ -91,7 +91,7 @@ async def login(self) -> str:
9191
"""
9292
return await self._api.login()
9393

94-
async def authenticate(self) -> tuple[str, str]:
94+
async def authenticate(self) -> Tuple[str, str]:
9595
"""
9696
Authenticate to conjur using credentials provided to credentials provider
9797
@return: API token

conjur_api/http/api.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
# Builtins
99
import logging
1010
from datetime import datetime
11-
from typing import Optional
11+
from typing import Optional, Tuple
1212
from urllib import parse
1313

1414
from conjur_api.errors.errors import HttpStatusError, InvalidResourceException, MissingRequiredParameterException
@@ -102,7 +102,7 @@ async def login(self) -> str:
102102
self.ssl_verification_data
103103
)
104104

105-
async def authenticate(self) -> tuple[str, datetime]:
105+
async def authenticate(self) -> Tuple[str, datetime]:
106106
"""
107107
Authenticate based on the chosen strategy to fetch a short-lived conjur_api token that
108108
for a limited time will allow you to interact fully with the Conjur vault.

requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# Keep this in sync with setup.py
22
nose2>=0.9.2
33
nose2[coverage_plugin]>=0.6.5
4-
pylint>=2.6.0
4+
pylint>=3.3.1
55
cryptography~=43.0.1
66
keyring>=23.0.0
77
pyopenssl>=20.0.0

0 commit comments

Comments
 (0)