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

Remove tty from serverless template #918

Merged
merged 3 commits into from
Jul 5, 2022
Merged

Remove tty from serverless template #918

merged 3 commits into from
Jul 5, 2022

Conversation

samchungy
Copy link
Contributor

@samchungy samchungy commented Jul 5, 2022

Noticed one of my repos working fine without this.

Looks like they disable interactivity when the process.env.CI environment variable is detected so this is no longer needed.

serverless/utils#162

@changeset-bot
Copy link

changeset-bot bot commented Jul 5, 2022

🦋 Changeset detected

Latest commit: 81d8228

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
skuba Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@samchungy samchungy marked this pull request as ready for review July 5, 2022 09:43
@samchungy samchungy requested review from a team as code owners July 5, 2022 09:43
@samchungy samchungy merged commit 2ba6d57 into master Jul 5, 2022
@samchungy samchungy deleted the tty branch July 5, 2022 13:03
@seek-oss-ci seek-oss-ci mentioned this pull request Jul 5, 2022
72636c added a commit that referenced this pull request Sep 10, 2022
* remove tty

* Create rare-walls-draw.md

Co-authored-by: Ryan Ling <ryan@outlook.com.au>
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.

None yet

2 participants