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

fix: show error stack or message like [email protected] #110

Merged
merged 1 commit into from
Aug 19, 2022
Merged

Conversation

stevenjoezhang
Copy link
Member

See

hexo-log/lib/log.js

Lines 48 to 52 in 214edf7

// Error
if (data.err) {
const err = data.err.stack || data.err.message;
if (err) msg += chalk.yellow(err) + '\n';
}

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.6%) to 98.387% when pulling d967f11 on dev into 2d03ea3 on master.

@stevenjoezhang stevenjoezhang merged commit b58e269 into master Aug 19, 2022
@stevenjoezhang stevenjoezhang deleted the dev branch August 19, 2022 03:32
@yoshinorin
Copy link
Member

Should we release a patch version(v3.1.1), or enough to ship with the next major version(4.0.0)?

@SukkaW
Copy link
Member

SukkaW commented Aug 19, 2022

Should we release a patch version(v3.1.1), or enough to ship with the next major version(4.0.0)?

I'd prefer a minor version since it is related to an existing issue.

@yoshinorin yoshinorin mentioned this pull request Aug 22, 2022
@yoshinorin
Copy link
Member

Just now I submit PR for release the 3.2.0 #111

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.

4 participants