Skip to content
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

Update docker container for CI #1393

Merged
merged 3 commits into from
Dec 2, 2023
Merged

Conversation

zaro0508
Copy link
Contributor

Update circleci docker container with python 3.7 removed and python 3.12 added

depends on Sceptre/sceptre-circleci#19

Update circleci docker container with python 3.7 removed
and python 3.12 added

depends on Sceptre/sceptre-circleci#19
.flake8 Show resolved Hide resolved
poetry.lock Outdated Show resolved Hide resolved
sceptre/stack.py Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
@iainelder
Copy link
Contributor

If the tests pass, then it looks good to me. I haven't had time to run these locally, but I trust the CI.

The custom image does have all the Python versions we expect.

$ docker run -it --rm sceptreorg/sceptre-circleci:2.1.0 bash -c 'for py in python3.{7,8,9,10,11,12}; do "$py" --version; done'
bash: line 1: python3.7: command not found
Python 3.8.18
Python 3.9.18
Python 3.10.6
Python 3.11.6
Python 3.12.0

@iainelder
Copy link
Contributor

I suppose we need to look at #1390 after this to finish adding support for Python 3.12. The main point here is to drop support for Python 3.7.

@zaro0508 zaro0508 merged commit f7b941f into Sceptre:master Dec 2, 2023
9 checks passed
zaro0508 pushed a commit that referenced this pull request Dec 3, 2023
Adds explicit support for Python 3.12. These are the minimal changes I needed for `pytest` to pass on 3.12 and macOS:

* Replaces deprecated [`datetime.datetime.utcnow`](https://docs.python.org/3/library/datetime.html#datetime.datetime.utcnow)
* Replaces `called_once_with` with `assert_called_once_with`, which now [raises](python/cpython#100690) `AttributeError: 'called_once_with' is not a valid assertion`
* Bumps `pytest` due to `DeprecationWarning: ast.Str is deprecated and will be removed in Python 3.14; use ast.Constant instead` pytest-dev/pytest#10977
* Refreshes the lockfile

depends on #1393
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants