Skip to content

Add "enabled" flag for SQS Offline#117

Closed
fernyettheplant wants to merge 1 commit into
CoorpAcademy:masterfrom
fernyettheplant:feat/add-enabled-flag
Closed

Add "enabled" flag for SQS Offline#117
fernyettheplant wants to merge 1 commit into
CoorpAcademy:masterfrom
fernyettheplant:feat/add-enabled-flag

Conversation

@fernyettheplant

Copy link
Copy Markdown

We had some cases that we wanted to do isolated functional tests but SQS was always required by it. With this flag, you can disable SQS offline if you need to run certain modules of your Serverless offline application in isolation.

@godu godu added the enhancement New feature or request label Jun 16, 2020

@AdrieanKhisbe AdrieanKhisbe left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me!

Would be interesting to have it applied to the other offline plugins.

I can take care if you want @jefer590 , along with rebasing to fix conflict, (and maybe little refactor (quick exit if (!enabled) return log();)

@AdrieanKhisbe AdrieanKhisbe self-assigned this Sep 25, 2020
@fernyettheplant

Copy link
Copy Markdown
Author

@AdrieanKhisbe thanks for the review
sure, go ahead :)

@macrouch

Copy link
Copy Markdown

What ever happened to this? This feature would be extremely useful

@silouone

Copy link
Copy Markdown
Member

Thank you for this, @fernyettheplant! Unfortunately the enabled flag was built against the plugin architecture that #265 (on master now, released shortly) reworked, so it no longer applies cleanly. The underlying need for opt-in/opt-out control is still valid and we're tracking it separately in #222. Closing this one, but thanks again for taking the time. 🙏

@silouone silouone closed this Jun 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants