-
Notifications
You must be signed in to change notification settings - Fork 7
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
chore(renovate): pin monaco-editor expressions
and entities-routes
#1354
base: main
Are you sure you want to change the base?
Conversation
renovate.json
Outdated
@@ -7,5 +7,6 @@ | |||
], | |||
"constraints": { | |||
"pnpm": "8.6.11" | |||
} | |||
}, | |||
"ignoreDeps": ["monaco-editor"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we just bump it for your specific package rather than for the entire repository?
It may be used in other places in the near future
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@adamdehaven Yes, that sounds legit to me. Would it be better if we only let the Renovate bot ignore monaco-editor
for @kong-ui-public/expressions
and @kong-ui-public/entities-routes
? (Would that be achievable via packageRules
?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, sounds good to me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we expedite this PR?
Seems it's needed to unblock upgrading @kong-ui-public/entities-routes
See example renovate pr failure https://github.com/Kong/konnect-ui-apps/actions/runs/8758729274
@mptap Sure. I've updated this PR to use Meanwhile, I don't feel the CI failure in the recent Renovate bumping PR (https://github.com/Kong/konnect-ui-apps/pull/3018) was caused by this change. Instead, it was because the Expressions editor component in |
@kong-ui-public/expressions
and @kong-ui-public/entities-routes
@kong-ui-public/expressions
and @kong-ui-public/entities-routes
expressions
and entities-routes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🚀
"packageRules": [ | ||
{ | ||
"matchPackageNames": ["@kong-ui-public/expressions", "@kong-ui-public/entities-routes"], | ||
"groupName": "exclude monaco-editor from @kong-ui-public/expressions and @kong-ui-public/entities-routes", | ||
"excludeDepNames": ["monaco-editor"] | ||
} | ||
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
issue: matchPackageNames
is for the matched dependency package names, not the packages within the repository
I think what you actually want here is something like matchFileNames
whereby you provide the path to the package.json
of these packages, e.g.
{
"packageRules": [
{
"description": "Pin monaco-editor dependency for the expressions package",
"matchFileNames": ["packages/core/expressions/package.json"],
"groupName": "expressions-monaco-editor",
"matchPackageNames": ["monaco-editor"],
"enabled": false
}
]
}
Summary
This PR tries to pin
monaco-editor
by excluding it for@kong-ui-public/expressions
and@kong-ui-public/entities-routes
viapackageRules
inrenovate.json
.Why?
monaco-editor
is currently only used for the Expressions routes support in@kong-ui-public/expressions
and@kong-ui-public/entities-routes
. We encountered several issues while consuming the latestmonaco-editor
in the host app during the development. To deliver the Expressions routes support in time, we decided to downgrade it tov0.21.3
–which we used previously in Kong Manager to prevent the Renovate bot from bumping it.We will try to update it to the latest possible minor release after Kong Gateway 3.7's release.