Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix(cz-git,cli): node v12 v14 startup error #200

Merged
merged 2 commits into from
Nov 5, 2024
Merged

fix(cz-git,cli): node v12 v14 startup error #200

merged 2 commits into from
Nov 5, 2024

Conversation

Zhengqbbb
Copy link
Owner

@Zhengqbbb Zhengqbbb commented Nov 5, 2024 •
edited
Loading

Related ISSUE

#199

Type Of Change

  • 🐞 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 📝 Document (This change requires a documentation update)
  • 🎨 Theme style (Theme style beautification)
  • ⚠ Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 🔨 Workflow (Workflow changes)

Clear Describe

  1. node-fetch using dynamic import. ensure node >= v12.20 can be work
  2. AI mode (need use node-fetch) add require node version >= 16.5 alert

Test Case

  • Test in Node.js >= v12.20 ~ <= 22 LTS

@Zhengqbbb
fix(cz-git): ensure >= node v12.20 can be work
14a4bca 
1. node-fetch using dynamic import. ensure node >= v12.20 can be work
2. AI mode add require node version >= 16.5
@Zhengqbbb
docs: ai mode add node version require
1952fc6
@netlify Netlify
Copy link

netlify bot commented Nov 5, 2024 •
edited
Loading

✅ Deploy Preview for cz-git ready!

Name Link
🔨 Latest commit 1952fc6
🔍 Latest deploy log https://app.netlify.com/sites/cz-git/deploys/672a0a441deec6000804ba73
😎 Deploy Preview https://deploy-preview-200--cz-git.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@Zhengqbbb Zhengqbbb mentioned this pull request Nov 5, 2024
Closed
@Zhengqbbb
Copy link
Owner Author

Node v12.20.0 Node v14.17.3

@Zhengqbbb Zhengqbbb merged commit a760f62 into main Nov 5, 2024
11 checks passed
@Zhengqbbb Zhengqbbb deleted the fix/node-version-require_199 branch November 5, 2024 12:39
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Zhengqbbb