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

Update all non-major dependencies #1274

Merged
merged 2 commits into from
Aug 3, 2022
Merged

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Jul 17, 2022

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
@types/estree 0.0.47 -> 0.0.52 age adoption passing confidence
@types/estree 0.0.51 -> 0.0.52 age adoption passing confidence
ace-builds 1.5.3 -> 1.8.1 age adoption passing confidence
acorn 8.7.0 -> 8.8.0 age adoption passing confidence
acorn 8.7.1 -> 8.8.0 age adoption passing confidence
astring 1.8.1 -> 1.8.3 age adoption passing confidence
gpu.js (source) 2.11.4 -> 2.15.2 age adoption passing confidence
jest-html-reporter 3.5.0 -> 3.6.0 age adoption passing confidence
jsdoc 3.6.10 -> 3.6.11 age adoption passing confidence
prettier (source) 2.6.2 -> 2.7.1 age adoption passing confidence
source-map 0.7.3 -> 0.7.4 age adoption passing confidence
typescript (source) 4.7.2 -> 4.7.4 age adoption passing confidence

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, click this checkbox. ⚠ Warning: custom changes will be lost.

This PR has been generated by Mend Renovate. View repository job log here.

@coveralls
Copy link

coveralls commented Jul 17, 2022

Pull Request Test Coverage Report for Build 2788790633

  • 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 85.149%

Totals Coverage Status
Change from base Build 2781290259: 0.0%
Covered Lines: 8071
Relevant Lines: 9233

💛 - Coveralls

@renovate renovate bot force-pushed the renovate/all-minor-patch branch from f37e3f9 to f99b24a Compare July 19, 2022 15:57
@renovate renovate bot changed the title Update dependency @types/estree to v0.0.52 Update all non-major dependencies Jul 19, 2022
@renovate renovate bot force-pushed the renovate/all-minor-patch branch 5 times, most recently from 298eae1 to ece24b5 Compare July 20, 2022 01:36
@shenyih0ng shenyih0ng added the dependencies Pull requests that update a dependency file label Jul 20, 2022
@renovate renovate bot force-pushed the renovate/all-minor-patch branch 21 times, most recently from 1bab465 to c51f94f Compare July 22, 2022 10:17
@renovate renovate bot force-pushed the renovate/all-minor-patch branch 22 times, most recently from 5263401 to 1e2f07c Compare August 3, 2022 02:07
@renovate renovate bot force-pushed the renovate/all-minor-patch branch from 1e2f07c to 74c76fe Compare August 3, 2022 05:15
@@ -25,7 +25,7 @@
"gpu.js": "^2.10.4",
"lodash": "^4.17.20",
"node-getopt": "^0.3.2",
"source-map": "^0.7.3",
"source-map": "0.7.3",
Copy link
Member

@shenyih0ng shenyih0ng Aug 3, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pin source-map version for now due to breaking change in new minor patch

Ref: mozilla/source-map#350

It changes the way source-map detects execution environment. New change will require refactor to how we handle tests js-slang (will make a PR addressing this).

Previously, it uses the presence of fetch api to determine if it is a browser env. After the change it checks for the presencewindow object. Since we run tests in a node env, tests that depends on source map (typically those that checks for error messages in code snippets) will fail.
We were able to pass tests previously because source-map interprets our testing environment as non-browser since it checks for fetch.

@shenyih0ng shenyih0ng requested a review from martin-henz August 3, 2022 10:55
Copy link
Member

@martin-henz martin-henz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good. tests pass and build works properly.

@shenyih0ng shenyih0ng merged commit e5647b5 into master Aug 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants