Skip to content

chore(quantic): octokit rule for quantic removed from renovate#3853

Merged
mmitiche merged 26 commits intomasterfrom
SFINT-5436
Apr 26, 2024
Merged

chore(quantic): octokit rule for quantic removed from renovate#3853
mmitiche merged 26 commits intomasterfrom
SFINT-5436

Conversation

@mmitiche
Copy link
Contributor

@mmitiche mmitiche commented Apr 25, 2024

SFINT-5436

This rule was previously added because a new major version of @octokit/graphql broke our implantation in Quantic script that uses this dependency.
This turned out to be an issue from their side that was fixed shortly after that new major version : octokit/graphql.js#551

Plus this renovate rules were containing a typo in the library name that prevented the rule from doing its job properly.
we had "matchPackagePatterns": ["^@otokit/graphql$"], instead of "matchPackagePatterns": ["^@octokit/graphql$"],

thus this dependency has been already updated by renovate to the latest version as you can see here:

"@octokit/graphql": "8.1.1",
"@octokit/graphql-schema": "15.12.0",

sallainCoveo and others added 25 commits February 26, 2024 10:49
* feat(commerce): get clientId from Relay instead of state

* refactor(event-protocol-utils): set getClientId method

* refactor(headless-commerce): use getVisitorId
@github-actions
Copy link

github-actions bot commented Apr 25, 2024

Pull Request Report

PR Title

✅ Title follows the conventional commit spec.

Bundle Size

File Old (kb) New (kb) Change (%)
case-assist 200.8 200.8 0
commerce 251.9 251.9 0
search 360.6 360.6 0
insight 341.6 341.6 0
product-listing 254.1 254.1 0
product-recommendation 171.3 171.3 0
recommendation 213.6 213.6 0
ssr 353.5 353.5 0

SSR Progress

Use case SSR (#) CSR (#) Progress (%)
search 39 44 89
recommendation 0 4 0
product-recommendation 0 10 0
product-listing 0 13 0
case-assist 0 6 0
insight 0 27 0
Detailed logs search : buildInteractiveResult
search : buildInteractiveInstantResult
search : buildInteractiveRecentResult
search : buildInteractiveCitation
search : buildGeneratedAnswer
recommendation : missing SSR support
product-recommendation : missing SSR support
product-listing : missing SSR support
case-assist : missing SSR support
insight : missing SSR support

@mmitiche mmitiche marked this pull request as ready for review April 25, 2024 11:06
@mmitiche mmitiche requested review from a team, louis-bompart and olamothe April 25, 2024 11:06
Copy link
Collaborator

@erocheleau erocheleau left a comment

Choose a reason for hiding this comment

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

So what you're saying is that it was working all along? 😂

@mmitiche
Copy link
Contributor Author

So what you're saying is that it was working all along? 😂

indeed x)

@mmitiche mmitiche added this pull request to the merge queue Apr 26, 2024
Merged via the queue into master with commit 0840fa1 Apr 26, 2024
@mmitiche mmitiche deleted the SFINT-5436 branch April 26, 2024 13:51
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.

4 participants