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

Improved error handling in git initialization #76506

Open
wants to merge 2 commits into
base: canary
Choose a base branch
from

Conversation

Devansh-001
Copy link

  • Added extra logs for easier error handling and mitigation.
  • Enhanced the error reporting in tryGitInit to log detailed messages when git initialization fails.
  • Ensured proper cleanup of .git directory if git init fails.
  • Added better handling of potential errors and unknown cases for debugging.

- Added extra logs for easier error handling and mitigation.
- Enhanced the error reporting in `tryGitInit` to log detailed messages when git initialization fails.
- Ensured proper cleanup of `.git` directory if `git init` fails.
- Added better handling of potential errors and unknown cases for debugging.
@ijjk ijjk added the create-next-app Related to our CLI tool for quickly starting a new Next.js application. label Feb 25, 2025
@ijjk
Copy link
Member

ijjk commented Feb 25, 2025

Allow CI Workflow Run

  • approve CI run for commit: 77d5082

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@Devansh-001
Copy link
Author

Thanks for looking into this! It looks like the CI workflow hasn’t been approved yet, which is causing the build to fail. Could you approve it when possible so I can debug properly?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
create-next-app Related to our CLI tool for quickly starting a new Next.js application.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants