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

エラーがスローされた際のメッセージを適切なものにする #33

Closed
camcam-lemon opened this issue Oct 16, 2018 · 3 comments
Assignees
Labels
discussion Discuss this issue
Milestone

Comments

@camcam-lemon
Copy link
Collaborator

try {
        const config = getConfig();
        const answers = await cz.prompt(config).catch(e => reject(e));
        resolve(
          commit(
            `${answers.prefix}${answers.scope}${answers.emoji}${answers.body}`
          )
        );
      } catch (e) {
        reject(e);
      }

rejectする際にエラーオブジェクトをそのまま渡しているので、どのようなエラーが表示されるのかがわからない。

  • promptの処理がどういう時に失敗するのか
  • commit()あるいはその他の処理が失敗した際のエラー処理をどうするか

上記の2点を調査して適切なエラーメッセージをrejectする仕様にするべき

@camcam-lemon camcam-lemon added the discussion Discuss this issue label Oct 16, 2018
@tyankatsu0105 tyankatsu0105 added this to the v1.0.0 milestone Oct 16, 2018
@tyankatsu0105
Copy link
Owner

ほげぇ

@tyankatsu0105 tyankatsu0105 modified the milestones: v1.0.0, v0.1.1 Oct 17, 2018
@tyankatsu0105
Copy link
Owner

@camcam-lemon
prompt,commitが失敗することってあるんですかね?
もうすでにこれらのメソッド定義でreject時のエラー文って発生されるようになってるかも??

@camcam-lemon
Copy link
Collaborator Author

@tyankatsu0105
正直わからんです。
中の実装見てみないことには正確にはどうなるかが・・・。

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

No branches or pull requests

2 participants