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 dependency 10up-toolkit to v4 #264

Merged
merged 2 commits into from
May 6, 2022
Merged

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented May 6, 2022

WhiteSource Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
10up-toolkit 3.1.2 -> 4.0.0 age adoption passing confidence

Release Notes

10up/10up-toolkit

v4.0.0

Compare Source

  • Changed: 10up-toolkit no longer transpiles @10up/block-components. Make sure to use the latest version of @10up/block-components that already ships transpiled code. #​181
  • Changed [BREAKING CHANGE]: Stop injecting wp-polyfill as dependecy of scripts built by 10up-toolkit. #​193.
  • Updated: Dependencies #​182
  • Changed [BREAKING CHANGE]: Stylelint and eslint are now peerDependencies. This is a breaking change only for those not using npm >= 7. [#​179] (https://github.com/10up/10up-toolkit/pull/179)

Configuration

📅 Schedule: 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.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, click this checkbox.

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

@jeffpaul jeffpaul added this to the 1.0.14 milestone May 6, 2022
@jeffpaul jeffpaul requested a review from cadic May 6, 2022 14:36
@jeffpaul
Copy link
Contributor

jeffpaul commented May 6, 2022

@cadic curious if any of the changes here will cause problems with the debug bar integration or if we're good to merge this into the core plugin?

Copy link
Contributor

@cadic cadic left a comment

Choose a reason for hiding this comment

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

@jeffpaul thank you for this upgrade. I see no conflicts with Sophi Debug Bar extension.

Testing this locally I found that npm run build fails in Node 12.

We still locked with Node 12 in this workflow, worth upgrading too:

https://github.com/globeandmail/sophi-for-wordpress/blob/5b1f8b84f0210b027de8a496566a61855a0de263/.github/workflows/dotorg-push-deploy.yml#L12-L15

@jeffpaul
Copy link
Contributor

jeffpaul commented May 6, 2022

@cadic probably bump to "node-version: '16'" yeah?

@sonarqubecloud
Copy link

sonarqubecloud bot commented May 6, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@cadic
Copy link
Contributor

cadic commented May 6, 2022

@jeffpaul I've pushed a commit with Node 16

@cadic cadic self-requested a review May 6, 2022 16:33
@jeffpaul jeffpaul merged commit a71bce1 into develop May 6, 2022
@jeffpaul jeffpaul deleted the renovate/10up-toolkit-4.x branch May 6, 2022 16:41
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.

3 participants