upgrade jest

[zhiyelee]

upgrade jest to latest version

[coveralls]

Pull Request Test Coverage Report for Build 7997322101

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 100.0%

---

Totals
Change from base Build 7979176448:	0.0%
Covered Lines:	98
Relevant Lines:	98

---

💛 - Coveralls

[arunshan]

@zhiyelee why did the dest/index.js change when you update jest? Should that be just a dev dependency?

[zhiyelee]

@zhiyelee why did the dest/index.js change when you update jest? Should that be just a dev dependency?

Good question. It's embarrassing that I forgot to update dest/index.js for the past two releases. The index.js will be used in the action instead of the source code. I realized this when working on this PR and decided to squeeze it in

[nickqi-tubi]

The upgrade looks good.

However, I noticed a warning in CI saying that "The set-output command is deprecated and will be disabled soon". So, we may need to upgrade @actions/core or @actions/github in a new PR as well.

[nickqi-tubi]

@zhiyelee why did the dest/index.js change when you update jest? Should that be just a dev dependency?

Good question. It's embarrassing that I forgot to update dest/index.js for the past two releases. The index.js will be used in the action instead of the source code. I realized this when working on this PR and decided to squeeze it in

Do we need to bump up the package version have a new NPM release?