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

feat(release): automate release via release-please #429

Merged
merged 10 commits into from
Feb 29, 2024

Conversation

totallyzen
Copy link
Collaborator

@totallyzen totallyzen commented Feb 27, 2024

changes

  • add release-please as the release automation tool
  • refactor "install python" into a reusable local action
  • build and test with Python 3.12 - seems it didn't take much extra effort

notes

  • actual "releases" here on GitHub is a mess
  • 3.7.1 is the latest on PyPI
  • only exists as a tag on this repo, no release notes
  • the latest "release" is put out later than some of the higher version tags
  • .github/.release-please-manifest.json -> went with the sha from 3.7.1 which then release-please will take as "latest live version", also made 3.7.1 in the manifest

This was referenced Feb 27, 2024
@totallyzen totallyzen self-assigned this Feb 27, 2024
inputs:
python-version:
description: "The python version to install and use"
default: "3.11" # we default to latest supported
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

... which is python 3.12: https://devguide.python.org/versions/

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not quite! :)
https://github.com/testcontainers/testcontainers-python/blob/main/pyproject.toml#L58-L59

BUT, this is definitely a no-brainer follow up once we've rebooted! Thanks for reading! ❤️

Copy link
Contributor

@jankatins jankatins Feb 28, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh fuuu... That should definitely be a <4.0 or even unbounded (I really dislike poetry in that regard, there are a lot of arguments that libraries should never use upper bounds in their own dependency spec).

Fun, we actually use testcontainers-python (or better all the "current" small packages) in a py3.12 project.

Really looking forward to get this rebooted and then a few of the "get rid of dependencies by looking at logs" PRs merged.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jankatins in the end it took no effort
a342b60

The project is running off of my and @alexanderankin 's excess energy.

Thanks for showing support, it adds a little extra energy to the mix, makes things easier.

@alexanderankin
Copy link
Collaborator

alexanderankin commented Feb 28, 2024 via email

@totallyzen totallyzen merged commit 30f859e into testcontainers:main Feb 29, 2024
7 checks passed
@totallyzen totallyzen deleted the feat/release-please-p1 branch February 29, 2024 14:13
@totallyzen totallyzen mentioned this pull request Feb 29, 2024
totallyzen pushed a commit that referenced this pull request Mar 6, 2024
🤖 I have created a release *beep* *boop*
---


##
[4.0.0](testcontainers-v3.7.1...testcontainers-v4.0.0)
(2024-03-06)

### Release Notes

The breaking changes are the ones we were able to easily track. If you
spot any new issues between `3.7.1` and `4.0.0`, please do report it and
we'll do our best to fix everything. The release is now

Some kudos from @totallyzen to folks who helped a great deal in starting
things again:
- kudos to @alexanderankin for his contribution on #426 
- kudos to @jankatins for feedback on various PRs including 
- kudos to @max-pfeiffer and @bearrito for their contributions as well


### ⚠ BREAKING CHANGES

* **compose:** implement compose v2 with improved typing
([#426](#426))
* **core:** add support for `tc.host` and de-prioritise `docker:dind`
([#388](#388))

### Features

* **build:** use poetry and organise modules
([#408](#408))
([6c69583](6c69583))
* **compose:** allow running specific services in compose
([f61dcda](f61dcda))
* **compose:** implement compose v2 with improved typing
([#426](#426))
([5356caf](5356caf))
* **core:** add support for `tc.host` and de-prioritise `docker:dind`
([#388](#388))
([2db8e6d](2db8e6d))
* **redis:** support AsyncRedisContainer
([#442](#442))
([cc4cb37](cc4cb37))
* **release:** automate release via release-please
([#429](#429))
([30f859e](30f859e))


### Bug Fixes

* Added URLError to exceptions to wait for in elasticsearch
([0f9ad24](0f9ad24))
* **build:** add `pre-commit` as a dev dependency to simplify local dev
and CI
([#438](#438))
([1223583](1223583))
* **build:** early exit strategy for modules
([#437](#437))
([7358b49](7358b49))
* changed files breaks on main
([#422](#422))
([3271357](3271357))
* flaky garbage collection resulting in testing errors
([#423](#423))
([b535ea2](b535ea2))
* rabbitmq readiness probe
([#375](#375))
([71cb75b](71cb75b))
* **release:** prove that the release process updates the version
([#444](#444))
([87b5873](87b5873))
* test linting issue
([427c9b8](427c9b8))


### Documentation

* Sphinx - Add title to each doc page
([#443](#443))
([750e12a](750e12a))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

None yet

3 participants