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

Support for RabbitMq versions 3.9 and 3.10 #40

Merged
merged 3 commits into from
Nov 24, 2022

Conversation

mryan43
Copy link

@mryan43 mryan43 commented Aug 26, 2022

  • Bumped default version in acceptance tests docker-compose.yml to 3.10
  • Added new jobs in travis.yml for version 3.9 and 3.10
  • Updated the readme to indicate that those versions are supported
  • Fixed a typo in word "versions" in readme

Ran acceptance tests locally for both 3.9 and 3.10 and they passed.

Versions 3.7 and 3.8 of rabbitmq are officially out of support : https://www.rabbitmq.com/versions.html#out-of-support
can we remove them from the list of officially supported versions for this provider ?

.travis.yml Outdated
@@ -13,6 +13,8 @@ env:
- RABBITMQ_USERNAME="guest"
- RABBITMQ_PASSWORD="guest"
jobs:
- RABBITMQ_VERSION=3.10
Copy link
Owner

Choose a reason for hiding this comment

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

Actually Travis is not used anymore, I should remove this file 🤦‍♂️
I removed it on master so you can merge it in your branch

README.md Outdated Show resolved Hide resolved
mryan43 and others added 2 commits August 29, 2022 09:43
@mryan43
Copy link
Author

mryan43 commented Aug 29, 2022

Requested changes pushed, please take a look again.

@mryan43
Copy link
Author

mryan43 commented Sep 15, 2022

Hello, any news on this ?

@Nielsoux
Copy link

Hello @cyrilgdn

Would you have some time to review this PR maybe?

Thanks in advance,
Niels

Copy link
Owner

@cyrilgdn cyrilgdn left a comment

Choose a reason for hiding this comment

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

Thanks 👍 , and sorry for the delay!

@cyrilgdn cyrilgdn merged commit 696ec6b into cyrilgdn:master Nov 24, 2022
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.

3 participants