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: better gracefully shutdown #1212

Merged
merged 1 commit into from
Jun 27, 2024
Merged

Conversation

Timeless0911
Copy link
Contributor

@Timeless0911 Timeless0911 commented Jun 27, 2024

Summary

Actively exit the process when gracefully shutdown.

Related Issue

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copy link

netlify bot commented Jun 27, 2024

Deploy Preview for aquamarine-blini-95325f ready!

Name Link
🔨 Latest commit e732da9
🔍 Latest deploy log https://app.netlify.com/sites/aquamarine-blini-95325f/deploys/667cdb8e96be670008db4890
😎 Deploy Preview https://deploy-preview-1212--aquamarine-blini-95325f.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 94 (🟢 up 5 from production)
Accessibility: 97 (no change from production)
Best Practices: 92 (no change from production)
SEO: 92 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@Timeless0911 Timeless0911 requested a review from chenjiahan June 27, 2024 03:02
@Timeless0911 Timeless0911 enabled auto-merge (squash) June 27, 2024 03:02
@Timeless0911 Timeless0911 force-pushed the chore/gracefully-shhutdown branch from 8dd989d to e732da9 Compare June 27, 2024 03:25
@Timeless0911 Timeless0911 changed the title chore: better gracefully shutdown feat: better gracefully shutdown Jun 27, 2024
@Timeless0911 Timeless0911 requested a review from chenjiahan June 27, 2024 03:29
@Timeless0911 Timeless0911 merged commit a68d86d into main Jun 27, 2024
9 checks passed
@Timeless0911 Timeless0911 deleted the chore/gracefully-shhutdown branch June 27, 2024 03:41
@@ -100,9 +100,14 @@ cli
await startDevServer();

const exitProcess = async () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

shall we add a timeout to force exit after 5 seconds?

Copy link
Member

Choose a reason for hiding this comment

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

5 seconds too too long

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants