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

[Snyk] Upgrade esformatter from 0.10.0 to 0.11.3 #18

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

snyk-bot
Copy link

Snyk has created this PR to upgrade esformatter from 0.10.0 to 0.11.3.

merge advice

✨What is Merge Advice? We check thousands of dependency upgrade pull requests and CI tests every day to see which upgrades were successfully merged. After crunching this data, we give a recommendation on how safe we think the change is for you to merge without causing issues. Learn more, and share your feedback to help improve this feature. 🙏

ℹ️ Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.


  • The recommended version is 4 versions ahead of your current version.
  • The recommended version was released a month ago, on 2020-03-17.

The recommended version fixes:

Severity Issue Exploit Maturity
Insecure use of Tmp files
npm:sync-exec:20160124
No Known Exploit
Release notes
Package name: esformatter
  • 0.11.3 - 2020-03-17
    • Updated dependencies
  • 0.11.2 - 2019-11-06
    • Updated dependencies
    • Fixes vulnerability issues with transitive dependencies
  • 0.11.1 - 2019-10-01

    0.11.1

  • 0.11.0 - 2019-10-01
    • BREAKING CHANGE: Only supports latest node LTS
    • Updated dependencies
  • 0.10.0 - 2016-10-30

    0.10.0

from esformatter GitHub release notes
Commit messages
Package name: esformatter

Compare


Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.

For more information:

🧐 View latest project report

🛠 Adjust upgrade PR settings

🔕 Ignore this dependency or unsubscribe from future upgrade PRs

@ci-reporter
Copy link

ci-reporter bot commented Apr 26, 2020

The build is failing

✨ Good work on this PR so far! ✨ Unfortunately, the Travis CI build is failing as of fa22569. Here's the output:

npm test
> jstransformer-esformatter@0.2.0 test /home/travis/build/jstransformers/jstransformer-esformatter
> test-jstransformer

 • esformatter
   ✓ transform has an output format (0ms)
   Output:	"var Comment = React.createClass(\n  {\n    render: function() {\n      return (\n        <div className=\"comment\">\n      <h2 className=\"commentAuthor\">\n        {this.props.author}\n      </h2>\n      {this.props.children}\n    </div>\n      );\n    }\n  });"
   Expected:	"var Comment = React.createClass(\n  {\n    render: function() {\n      return (\n        <div className=\"comment\">\n      <h2 className=\"commentAuthor\">\n        {this.props.author}\n      </h2>\n      {this.props.children}\n    </div>\n        );\n    }\n  });"
   ✗ esformatter.render() (28ms)

     AssertionError [ERR_ASSERTION]: 'var Comment = React.createClass(\n  {\n    render: function() {\n      return (\n        <div className="comment">\n      <h2 c === 'var Comment = React.createClass(\n  {\n    render: function() {\n      return (\n        <div className="comment">\n      <h2 c
         at assertEqual (/home/travis/build/jstransformers/jstransformer-esformatter/node_modules/test-jstransformer/index.js:10:12)
         at checkOutput (/home/travis/build/jstransformers/jstransformer-esformatter/node_modules/test-jstransformer/index.js:103:9)
         at TestCase.test [as fn] (/home/travis/build/jstransformers/jstransformer-esformatter/node_modules/test-jstransformer/index.js:140:9)
         at /home/travis/build/jstransformers/jstransformer-esformatter/node_modules/testit/lib/suite.js:74:29
         at tryCallOne (/home/travis/build/jstransformers/jstransformer-esformatter/node_modules/promise/lib/core.js:37:12)
         at /home/travis/build/jstransformers/jstransformer-esformatter/node_modules/promise/lib/core.js:123:15
         at flush (/home/travis/build/jstransformers/jstransformer-esformatter/node_modules/asap/raw.js:50:29)
         at _combinedTickCallback (internal/process/next_tick.js:132:7)
         at process._tickCallback (internal/process/next_tick.js:181:9)

Total duration 37ms
tests failed

I'm sure you can fix it! If you need help, don't hesitate to ask a maintainer of the project!


This comment was automagically generated by ci-reporter. If you see a problem, open an issue here.

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.

None yet

1 participant