Skip to content

fix: polish and improve postinstall onboarding message #7248

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

Merged
merged 4 commits into from
May 1, 2025

Conversation

serhalp
Copy link
Collaborator

@serhalp serhalp commented May 1, 2025

Summary

  • Use a pretty Netlify-cyan box with the Netlify diamond icon
  • Mention ntl login as a likely first step
  • Link to better CLI entrypoint URL
  • Link to main Netlify docs

Before

Screenshot 2025-04-30 at 13 15 13

After

Screenshot 2025-04-30 at 13 32 07

Copy link

github-actions bot commented May 1, 2025

📊 Benchmark results

Comparing with 841d782

  • Dependency count: 1,150 (no change)
  • Package size: 280 MB (no change)
  • Number of ts-expect-error directives: 404 (no change)

serhalp added 2 commits May 1, 2025 07:59
- Use a pretty cyan box with the Netlify diamond icon
- Mention `ntl login` as a likely first step
- Link to better CLI entrypoint URL
- Link to main Netlify docs
@serhalp serhalp force-pushed the fix/cpla-2543-postinstall-polish branch from a0ecf05 to 7f27116 Compare May 1, 2025 11:59
@serhalp serhalp marked this pull request as ready for review May 1, 2025 12:08
@serhalp serhalp requested a review from a team as a code owner May 1, 2025 12:08
@serhalp serhalp enabled auto-merge (squash) May 1, 2025 16:12
@serhalp serhalp merged commit d9523c7 into main May 1, 2025
52 checks passed
@serhalp serhalp deleted the fix/cpla-2543-postinstall-polish branch May 1, 2025 20:10
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