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

Prepare aws-sdk-rails 4.2 modularization #164

Merged
merged 5 commits into from
Nov 19, 2024
Merged

Prepare aws-sdk-rails 4.2 modularization #164

merged 5 commits into from
Nov 19, 2024

Conversation

mullermp
Copy link
Contributor

This change uses the modular gems and fixes up the CI to make the release tools work.

@mullermp
Copy link
Contributor Author

@ssunday If you are able, could you please pull this branch, and test it in your application? Can you also specify which features you are using? This branch should pull in the newly published modular gems (there are 5 of them). Everything in this major version should be backwards compatible. If it's not, please let me know what sharp edges you run into and I can put out some fixes. I plan to make breaking changes on the modular gems ~> 1 (they are published as 0.1.0 now) and on aws-sdk-rails ~> 5.

@mullermp
Copy link
Contributor Author

Any bravehearts are welcome to also do some testing..

@ssunday
Copy link
Contributor

ssunday commented Nov 18, 2024

Haven't done any testing, but it seems like one day this gem can go away if the rails specific logic becomes optional in each gems (so they are not rails specific but have the rails feature set). Like it's weird to have to pull in the omnibus gem if you only want like the sqs functionality, but if you use EB you also need this...etc.

Will try to test it out soon on a project!

@mullermp
Copy link
Contributor Author

The middleware can technically be used with any adapter, not just sqs. That's why it belongs here imo. Also, some generic features like encrypted credentials, notifications, rails logger as SDK logger, etc, belong here and did not make sense to split out.

I appreciate volunteering to test! I did manual tests and unit tests on my own but it doesn't hurt to be sure.

@ssunday
Copy link
Contributor

ssunday commented Nov 18, 2024

Makes sense!

@mullermp mullermp merged commit 008521f into main Nov 19, 2024
15 checks passed
@mullermp mullermp deleted the 4.2 branch November 19, 2024 22:26
@ssunday
Copy link
Contributor

ssunday commented Dec 4, 2024

Been looking good so far

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