-
Notifications
You must be signed in to change notification settings - Fork 14
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(deps): ignore clients package #535
Conversation
✅ Deploy Preview for api-clients-automation canceled.
|
✗ The generated branch has been deleted.If the PR has been merged, you can check the generated code on the |
renovate.json
Outdated
"**/client-abtesting/**", | ||
"**/client-analytics/**", | ||
"**/client-insights/**", | ||
"**/client-personalisation/**", | ||
"**/client-predict/**", | ||
"**/client-query-suggestions/**", | ||
"**/client-search/**", | ||
"**/client-sources/**", | ||
"**/recommend/**" |
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.
Does it take negative patterns? Might be easier to only ignore manually written clients
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.
clients/algoliasearch-client-javascript/package.json
clients/algoliasearch-client-javascript/packages/algoliasearch-lite/package.json
clients/algoliasearch-client-javascript/packages/algoliasearch/package.json
clients/algoliasearch-client-javascript/packages/client-abtesting/package.json
clients/algoliasearch-client-javascript/packages/client-analytics/package.json
clients/algoliasearch-client-javascript/packages/client-common/package.json
clients/algoliasearch-client-javascript/packages/client-insights/package.json
clients/algoliasearch-client-javascript/packages/client-personalization/package.json
clients/algoliasearch-client-javascript/packages/client-predict/package.json
clients/algoliasearch-client-javascript/packages/client-query-suggestions/package.json
clients/algoliasearch-client-javascript/packages/client-search/package.json
clients/algoliasearch-client-javascript/packages/client-sources/package.json
clients/algoliasearch-client-javascript/packages/recommend/package.json
clients/algoliasearch-client-javascript/packages/requester-browser-xhr/package.json
clients/algoliasearch-client-javascript/packages/requester-node-http/package.json
eslint/package.json
package.json
playground/javascript/browser/package.json
playground/javascript/node/package.json
scripts/package.json
website/package.json
tests/output/javascript/package.json
10 generated package.json
and 12 hand written !
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.
the tests package is already ignored for some reason
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.
I meant for the client JS only, like
!clients/algoliasearch-client-javascript/packages/requester-*/**
!clients/algoliasearch-client-javascript/packages/client-common/**
So later we don't have to worry if we add JS clients
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.
I can try, it doesn't says if it's supported in the doc
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.
No problem then! Also you can add algoliasearch
and algoliasearch-lite
to the list if it does not work
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.
it has to be merged to try 😁
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.
let's see!
🧭 What and Why
Renovate is trying to update the deps in our generated clients, we don't want that.