Workflows: Try different SHA

[Mamaduka]

What?

Related #70023.

Try different SHA for the performance tests workflow. The current choice might have been too recent, and I don't see new commits in https://www.codevitals.run/project/gutenberg.

How?

See https://developer.wordpress.org/block-editor/explanations/architecture/performance/#update-the-reference-commit.

Testing Instructions

These changes are hard to test without merging.

[github-actions]

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: Mamaduka <mamaduka@git.wordpress.org>
Co-authored-by: t-hamano <wildworks@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

[t-hamano]

From what I can see, it looks like a request error is occurring here:

gutenberg/bin/log-performance-results.js

Lines 88 to 94 in bcc41b1

	const req = https.request( options, ( res ) => {
	console.log( `statusCode: ${ res.statusCode }` );
	res.on( 'data', ( d ) => {
	process.stdout.write( d );
	} );
	} );

Normally, a successful HTTP request to the code health should get a response with a 200 status code:

However, after updating the base commit, we are getting a 405 error, which means the request is not allowed for the resource:

Is this issue related to #69126? Maybe adding the appropriate permissions to that job will solve it?

[Mamaduka]

Is this issue related to #69126? Maybe adding the appropriate permissions to that job will solve it?

I can see commits tracked in CodeVitals after this PR, but double-checking won't hurt.

cc @johnbillion, @desrosj

[t-hamano]

One more thing I was wondering, does this SHA need to be updated?

gutenberg/.github/workflows/performance.yml

Line 112 in bcc41b1

./bin/log-performance-results.js "$CODEHEALTH_PROJECT_TOKEN" trunk "$GITHUB_SHA" c7722262e65a3f4d0f1a2d1ad29eccb2069509e4 "$COMMITTED_AT"

It appears that in the past, when the minimum required WordPress version has been bumped, this commit hash has been updated as well.

• Bump minimum required WordPress version to 6.6: https://github.com/WordPress/gutenberg/pull/67117/files#diff-f85093d858e907df98c439a06dfc23c61972489c29b57014d6c4d7d2251b1a36L104
• Bump minimum required WordPress version to 6.5: https://github.com/WordPress/gutenberg/pull/64126/files#diff-f85093d858e907df98c439a06dfc23c61972489c29b57014d6c4d7d2251b1a36L104

I noticed that #70023 didn't update this line 🤔

[Mamaduka]

Great catch, @t-hamano! I did a search and replace, but it looks like I missed the spot.

[t-hamano]

Let's give it a try 👍

[Mamaduka]

This worked. Thanks for catching the missing SHA replacement, @t-hamano!