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: added support for sqs-consumer v8, v9 #1075

Merged
merged 1 commit into from
Mar 18, 2024
Merged

Conversation

kirrg001
Copy link
Contributor

refs 158386

@@ -209,7 +209,7 @@
"sinon-chai": "^3.7.0",
"socket.io": "^4.6.0",
"socket.io-client": "^4.6.0",
"sqs-consumer": "^7.4.0",
"sqs-consumer": "^9.1.0",
"sqs-consumer-v5": "npm:sqs-consumer@^5.7.0",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

NOTE: sqs-consumer-v5 is used by aws v2.

@kirrg001 kirrg001 force-pushed the update-sqs-consumer branch from dac3b53 to 105460f Compare March 14, 2024 14:20
@kirrg001
Copy link
Contributor Author

v8 is already broken

@kirrg001
Copy link
Contributor Author

The underlying problem is #1076

@kirrg001 kirrg001 force-pushed the update-sqs-consumer branch from 105460f to dd2fa98 Compare March 15, 2024 13:05
@kirrg001 kirrg001 marked this pull request as ready for review March 15, 2024 13:32
@kirrg001 kirrg001 requested a review from a team as a code owner March 15, 2024 13:32
@kirrg001
Copy link
Contributor Author

@aryamohanan We can ship that on Monday as well.

@aryamohanan aryamohanan merged commit 832238e into main Mar 18, 2024
1 of 2 checks passed
@aryamohanan aryamohanan deleted the update-sqs-consumer branch March 18, 2024 05:20
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