-
Notifications
You must be signed in to change notification settings - Fork 8
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
Dependency Dashboard #1537
Comments
# Pull Request ## 🤨 Rationale [Milan pointed out](#1542 (comment)) Renovate will fail to update Playwright because: 1. there is no Nuget version corresponding to the latest NPM version 2. there was a breaking change between 1.37 and 1.38 This will block automatic updates for all of our devDependencies. ## 👩💻 Implementation 1. Remove Playwright from the list of dev dependencies updated by Renovate 2. Filed tech debt #1544 to address this ## 🧪 Testing Once this goes in I'll inspect #1537 to ensure it no longer lists Playwright ## ✅ Checklist <!--- Review the list and put an x in the boxes that apply or ~~strike through~~ around items that don't (along with an explanation). --> - [x] I have updated the project documentation to reflect my changes or determined no changes are needed.
# Pull Request ## 🤨 Rationale Now that we established Renovate to update nuget deps in #1675 I noticed on the dependency dashboard #1537 that the 3 bunit dependencies will be updated in separate PRs. We want them to be updated together. Here's what the dependency dashboard showed before this change: --- ![image](https://github.com/ni/nimble/assets/10500124/044dbb45-e10a-4359-8e2e-b7b9a64356f3) --- ## 👩💻 Implementation Add a package rule for any nuget package that starts with "bunit". I considered grouping all minor/patch nuget dependencies in one group to reduce the PR noise, similar to what we did for npm. We may yet decide to do this but I wanted to get some runtime with the default configuration and hear the team's feedback first. ## 🧪 Testing I will inspect the dashboard when this completes to ensure it works as intended. ## ✅ Checklist <!--- Review the list and put an x in the boxes that apply or ~~strike through~~ around items that don't (along with an explanation). --> - [ ] I have updated the project documentation to reflect my changes or determined no changes are needed.
@TrevorKarjanis curious if there was a reason to unpin this issue, is it showing up somewhere else when pinned? |
Uhhh I was on this repository by accident, because I meant to go to the style guide. I must have accidentally clicked the unpin button, because I have no idea what this issue is. I didn't even know where the pin/unpin button was until you brought it up. 😁 |
Marking triage so we can discuss owners for closing dep upgrades |
Renovate notes:
|
This issue lists Renovate updates and detected dependencies. Read the Dependency Dashboard docs to learn more.
Config Migration Needed
Awaiting Schedule
These updates are awaiting their schedule. Click on a checkbox to get an update now.
@angular-devkit/build-angular
,@angular/cli
,@ni/eslint-config-angular
)Microsoft.AspNetCore.Components.WebView.Wpf
,bunit
,bunit.core
,bunit.web
)@storybook/addon-a11y
,@storybook/addon-actions
,@storybook/addon-docs
,@storybook/addon-essentials
,@storybook/addon-interactions
,@storybook/addon-links
,@storybook/cli
,@storybook/html
,@storybook/html-webpack5
,@storybook/manager-api
,@storybook/theming
,@tiptap/core
,@tiptap/extension-bold
,@tiptap/extension-bullet-list
,@tiptap/extension-document
,@tiptap/extension-hard-break
,@tiptap/extension-history
,@tiptap/extension-italic
,@tiptap/extension-link
,@tiptap/extension-list-item
,@tiptap/extension-mention
,@tiptap/extension-ordered-list
,@tiptap/extension-paragraph
,@tiptap/extension-placeholder
,@tiptap/extension-text
,@types/node
,beachball
,concurrently
,rollup
,storybook
,tslib
,vite
,webpack
)@11ty/eleventy
,@chromatic-com/storybook
,glob
,rimraf
)Microsoft.NET.Test.Sdk
,coverlet.collector
)Ignored or Blocked
These are blocked by an existing closed PR and will not be recreated unless you click a checkbox below.
Detected dependencies
github-actions
npm
nuget
The text was updated successfully, but these errors were encountered: