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

CI housekeeping #104

Merged
merged 1 commit into from
Aug 6, 2024
Merged

CI housekeeping #104

merged 1 commit into from
Aug 6, 2024

Conversation

kianmeng
Copy link
Contributor

@kianmeng kianmeng commented Aug 5, 2024

List of changes:

  • add Elixir 1.17 / Erlang OTP 27 to CI
  • bump GitHub actions

mix.exs Outdated
@@ -9,7 +9,7 @@ defmodule BroadwayCloudPubSub.MixProject do
[
app: :broadway_cloud_pub_sub,
version: @version,
elixir: "~> 1.11",
elixir: "~> 1.12",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Bump to 1.12 where the security support just ended recently, see https://endoflife.date/elixir

Not sure if we should sync with Broadway min Elixir requirement, which is still set to 1.7, see https://github.com/dashbitco/broadway/blob/9eaf2140b6c1f36b25f990a7c74052bece26aadc/mix.exs#L11

Copy link
Member

Choose a reason for hiding this comment

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

It is end of life but we should not be the ones forcing users to update. If we can still support 1.11, then it should. :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Noted, and updated.

List of changes:
- add Elixir 1.17 / Erlang OTP 27 to CI
- bump github actions
@josevalim josevalim merged commit c8f62bf into dashbitco:main Aug 6, 2024
2 checks passed
@josevalim
Copy link
Member

💚 💙 💜 💛 ❤️

@kianmeng
Copy link
Contributor Author

kianmeng commented Aug 7, 2024

🥳 🥳 🥳 🥳 🥳

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