Closed chengcyber closed 2 months ago
Have you run rush change for this change?
rush change
If No, please run rush change before, this is necessary.
If adding a new feature, the PR's description includes:
Does this PR introduce a breaking change? (check one)
If yes, please describe the impact and migration path for existing applications:
Patch lunr.utils.warn to use console.log instead of console.warn
lunr.utils.warn
console.log
console.warn
Lunr prints error message with console.warn which print on stderr. The messages are treated as error message by rush build
rush build
After patching the code of lunr, it uses console.log to print the message.
Local
Basic Checks
Have you run
rush changefor this change?If No, please run
rush changebefore, this is necessary.If adding a new feature, the PR's description includes:
Does this PR introduce a breaking change? (check one)
If yes, please describe the impact and migration path for existing applications:
Summary
Patch
lunr.utils.warnto useconsole.loginstead ofconsole.warnDetail
Lunr prints error message with
console.warnwhich print on stderr. The messages are treated as error message byrush buildAfter patching the code of lunr, it uses
console.logto print the message.How to test it
Local