Skip to content

Comments

run codespell pre-commit hook on staged files only#29360

Closed
quarckster wants to merge 1 commit intoopenssl:masterfrom
quarckster:pre-commit-codespell
Closed

run codespell pre-commit hook on staged files only#29360
quarckster wants to merge 1 commit intoopenssl:masterfrom
quarckster:pre-commit-codespell

Conversation

@quarckster
Copy link
Contributor

Please backport to all active branches.

@quarckster quarckster self-assigned this Dec 9, 2025
@quarckster quarckster moved this to Waiting Review in Development Board Dec 9, 2025
@mattcaswell mattcaswell added branch: master Applies to master branch approval: review pending This pull request needs review by a committer triaged: cleanup The issue/pr deals with cleanup of comments/docs not altering code significantly branch: 3.0 Applies to openssl-3.0 branch tests: exempted The PR is exempt from requirements for testing branch: 3.3 Applies to openssl-3.3 branch: 3.4 Applies to openssl-3.4 branch: 3.5 Applies to openssl-3.5 branch: 3.6 Applies to openssl-3.6 labels Dec 9, 2025
@github-project-automation github-project-automation bot moved this from Waiting Review to Waiting Merge in Development Board Dec 9, 2025
@t8m t8m added approval: done This pull request has the required number of approvals triaged: bug The issue/pr is/fixes a bug and removed approval: review pending This pull request needs review by a committer labels Dec 9, 2025
@openssl-machine openssl-machine added approval: ready to merge The 24 hour grace period has passed, ready to merge and removed approval: done This pull request has the required number of approvals labels Dec 10, 2025
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

openssl-machine pushed a commit that referenced this pull request Dec 10, 2025
Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Neil Horman <[email protected]>
(Merged from #29360)
openssl-machine pushed a commit that referenced this pull request Dec 10, 2025
Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Neil Horman <[email protected]>
(Merged from #29360)

(cherry picked from commit d70b22b)
openssl-machine pushed a commit that referenced this pull request Dec 10, 2025
Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Neil Horman <[email protected]>
(Merged from #29360)

(cherry picked from commit d70b22b)
openssl-machine pushed a commit that referenced this pull request Dec 10, 2025
Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Neil Horman <[email protected]>
(Merged from #29360)

(cherry picked from commit d70b22b)
@nhorman
Copy link
Contributor

nhorman commented Dec 10, 2025

merged to all active branches, thank you!

@nhorman nhorman closed this Dec 10, 2025
@github-project-automation github-project-automation bot moved this from Waiting Merge to Done in Development Board Dec 10, 2025
openssl-machine pushed a commit that referenced this pull request Dec 10, 2025
Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Neil Horman <[email protected]>
(Merged from #29360)

(cherry picked from commit d70b22b)
openssl-machine pushed a commit that referenced this pull request Dec 10, 2025
Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Neil Horman <[email protected]>
(Merged from #29360)

(cherry picked from commit d70b22b)
cxx194832 pushed a commit to cxx194832/openssl that referenced this pull request Dec 12, 2025
Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Neil Horman <[email protected]>
(Merged from openssl#29360)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Applies to master branch branch: 3.0 Applies to openssl-3.0 branch branch: 3.3 Applies to openssl-3.3 branch: 3.4 Applies to openssl-3.4 branch: 3.5 Applies to openssl-3.5 branch: 3.6 Applies to openssl-3.6 tests: exempted The PR is exempt from requirements for testing triaged: bug The issue/pr is/fixes a bug triaged: cleanup The issue/pr deals with cleanup of comments/docs not altering code significantly

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants