[Bug report] Incorrect y/N prompt

[sagargurtu]

✅ Checklist

• I am using the latest version
• I have searched FAQ

📄 System Info

System:
    OS: macOS 14.4.1
    CPU: (10) arm64 Apple M1 Pro
    Memory: 340.53 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.20.2 - ~/.nvm/versions/node/v18.20.2/bin/node
    npm: 10.5.0 - ~/.nvm/versions/node/v18.20.2/bin/npm
  npmPackages:
    commitizen: ^4.3.0 => 4.3.0 
    cz-git: ^1.9.2 => 1.9.2

🔎 Description

Concise Description

The confirmCommit prompt shows "Yneh".

Expected behavior

It should show y/N instead.

🔗 Reproduction link / repo:

No response

[Zhengqbbb]

It should show y/N instead.

I don't think there is an error. If the result meets expectations, I hope it can be submitted directly by Enter.
By the way, if you use h, you will understand why it's 'Yneh'.

---

Or you can explain what's error with the default value
Or you can explain what's wrong with the default value.

[sagargurtu]

Oh gosh, my terminal isn't showing the options (y,n,e,h) below the confirmCommit prompt and I was wondering what that was. Resolving.

[sagargurtu]

Here's the screenshot.

[Zhengqbbb]

Hey man, just h ^{show help} + Enter

CleanShot.2024-05-29.at.15.39.56.mp4

[marios1861]

When selecting e, how can we customize the editor that will be used? I don't want to use vi. thanks

[marios1861]

I found out how to do it. In cz-git/packages/src/shared/utils/editor.ts in the function editor, there is this line of choices:

const editor = opts.editor || process.env.VISUAL || process.env.EDITOR || ed

where ed is vim on unix-like systems. If we change the script declaration in package.json:

"scripts": {
    "commit": "EDITOR='code --wait' cz"
}

The process.env.EDITOR option will be picked up. This is definitely missing from the docs and it would be very helpful so that others don't embark on a similar wild goose chase. Thanks for the package.

[Zhengqbbb]

The process.env.EDITOR option will be picked up. This is definitely missing from the docs and it would be very helpful so that others don't embark on a similar wild goose chase. Thanks for the package.

I will add it in the FAQ docs, thank you for your support and pointing out this problem.

[Zhengqbbb]

@marios1861 The process.env.EDITOR option will be picked up. This is definitely missing from the docs and it would be very helpful so that others don't embark on a similar wild goose chase. Thanks for the package.

Hi, The latest version 🌟(v1.10.0)🌟 add env CZ_EDITOR to control what EDITOR use. Avoid affecting others or env.

please have a try 💪 and check document information
https://cz-git.qbb.sh/faq/#modifying-the-editor-editor-used-when-choosing-to-edit-during-commit-confirmation-yneh

[marios1861]

Hi! Lovely work!! I would only add a link to the cross-env docs to keep it self contained. Thanks for the package 👑.