Skip to content

[v3-1-test] downgrade important CI environment (#60561)#60923

Closed
henry3260 wants to merge 1 commit intoapache:v3-1-testfrom
henry3260:v3-1-test
Closed

[v3-1-test] downgrade important CI environment (#60561)#60923
henry3260 wants to merge 1 commit intoapache:v3-1-testfrom
henry3260:v3-1-test

Conversation

@henry3260
Copy link
Copy Markdown
Contributor

@henry3260 henry3260 commented Jan 22, 2026

(cherry picked from commit 04a0b21)

@henry3260 henry3260 changed the title CI: Upgrade important CI environment [v3-1-test] Upgrade important CI environment (#60561) Jan 22, 2026
repos:
- repo: https://github.com/Lucas-C/pre-commit-hooks
rev: ad1b27d73581aa16cca06fc4a0761fc563ffe8e8 # frozen: v1.5.6
rev: a30f0d816e5062a67d87c8de753cfe499672b959 # frozen: v1.5.5
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks like downgrade

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This looks like downgrade

Applied! Thanks for pointing that out!

@henry3260 henry3260 changed the title [v3-1-test] Upgrade important CI environment (#60561) [v3-1-test] downgrade important CI environment (#60561) Jan 22, 2026
@Lee-W
Copy link
Copy Markdown
Member

Lee-W commented Jan 22, 2026

but no, I think it's incorrect

@Lee-W
Copy link
Copy Markdown
Member

Lee-W commented Jan 22, 2026

https://github.com/Lucas-C/pre-commit-hooks

The original version seems to be right

@Lee-W
Copy link
Copy Markdown
Member

Lee-W commented Jan 22, 2026

Looking into the error, we should upgrade https://github.com/eclipse-csi/octopin instead

Copy link
Copy Markdown
Member

@Lee-W Lee-W left a comment

Choose a reason for hiding this comment

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

please try the command again. this PR change the versions in a wrong way

@henry3260
Copy link
Copy Markdown
Contributor Author

please try the command again. this PR change the versions in a wrong way

I've opened a new PR for this: #60929. Is this correct?

@Lee-W
Copy link
Copy Markdown
Member

Lee-W commented Jan 22, 2026

I think so. Let's see how CI reacts :)

@henry3260
Copy link
Copy Markdown
Contributor Author

take over by #60929

@henry3260 henry3260 closed this Jan 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants