Skip to content

Commit 43f5eb6

Browse files
authored
chore: [sc-71013] Drop future lib dependency (#118)
* Drop future library dependency * Update library version * Remove codeclimate steps from CI * Fix python syntax error
1 parent 07e0aaf commit 43f5eb6

File tree

5 files changed

+9
-33
lines changed

5 files changed

+9
-33
lines changed

.github/workflows/test.yml

Lines changed: 1 addition & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -50,32 +50,5 @@ jobs:
5050
run: |
5151
python -m pip install --upgrade pip setuptools wheel coverage
5252
pip install -e .[testing]
53-
- name: Set ENV for codeclimate (pull_request)
54-
continue-on-error: true
55-
run: |
56-
git fetch --no-tags --prune --depth=1 origin +refs/heads/$GITHUB_HEAD_REF:refs/remotes/origin/$GITHUB_HEAD_REF
57-
echo "GIT_BRANCH=$GITHUB_HEAD_REF" >> $GITHUB_ENV
58-
echo "GIT_COMMIT_SHA=$(git rev-parse origin/$GITHUB_HEAD_REF)" >> $GITHUB_ENV
59-
if: github.event_name == 'pull_request'
60-
- name: Set ENV for codeclimate (push)
61-
run: |
62-
echo "GIT_BRANCH=${GITHUB_REF#refs/heads/}" >> $GITHUB_ENV
63-
echo "GIT_COMMIT_SHA=$GITHUB_SHA" >> $GITHUB_ENV
64-
if: github.event_name == 'push'
65-
- name: Install Code Climate test report
66-
run: |
67-
curl -L https://codeclimate.com/downloads/test-reporter/test-reporter-latest-linux-amd64 > ./cc-test-reporter
68-
chmod +x ./cc-test-reporter
69-
./cc-test-reporter before-build
7053
- name: Run tests
71-
run: coverage run -m unittest && coverage xml -i --include='chartmogul/*'
72-
- name: Send Report to Code Climate
73-
if: ${{ success() }}
74-
env:
75-
CC_TEST_REPORTER_ID: ${{ secrets.CC_TEST_REPORTER_ID }}
76-
run: |
77-
if [ -z "$CC_TEST_REPORTER_ID" ]; then
78-
echo "⚠️ Skipping Code Climate upload — CC_TEST_REPORTER_ID is not set."
79-
else
80-
./cc-test-reporter after-build -t coverage.py
81-
fi
54+
run: coverage run -m unittest && coverage xml -i --include='chartmogul/*'

CHANGELOG.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,12 @@ and this project adheres to [Semantic Versioning].
88
[Keep a Changelog]: https://keepachangelog.com/en/1.0.0/
99
[Semantic Versioning]: https://semver.org/spec/v2.0.0.html
1010

11+
## [4.6.3] - 2025-09-01
12+
- Remove future dependency to resolve vulnerability issues
13+
14+
## [4.6.2] - 2025-07-09
15+
- Update Marshmallow dependency to use >=3.24.0
16+
1117
## [4.6.1] - 2025-05-19
1218
- Fixed Tasks API schema issue
1319
- Unify requirements in a single place

chartmogul/errors.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
from future.utils import raise_from
21
from requests import HTTPError
32

43

@@ -20,6 +19,6 @@ class DeprecatedArgumentError(Exception):
2019

2120
def annotateHTTPError(err):
2221
if isinstance(err, HTTPError):
23-
raise_from(APIError(err.response.content), err)
22+
raise APIError(err.response.content) from err
2423
else:
2524
raise err

chartmogul/version.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
__version__ = "4.6.2"
1+
__version__ = "4.6.3"

setup.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
#!/usr/bin/env python
2-
from __future__ import absolute_import
32
import os
43
import re
54
import sys
@@ -18,7 +17,6 @@
1817
"uritemplate>=4.1.1",
1918
"promise>=2.3.0",
2019
"marshmallow>=3.24.0,<5.0.0",
21-
"future>=0.18.3",
2220
"urllib3>=2.2.2",
2321
]
2422
test_requirements = [

0 commit comments

Comments
 (0)