Skip to content

Update GitHub actions as suggested by dependabot#19581

Closed
t8m wants to merge 1 commit into
openssl:masterfrom
t8m:gh-actions-version-update
Closed

Update GitHub actions as suggested by dependabot#19581
t8m wants to merge 1 commit into
openssl:masterfrom
t8m:gh-actions-version-update

Conversation

@t8m
Copy link
Copy Markdown
Member

@t8m t8m commented Nov 2, 2022

This should be trivially cherry-pickable to older branches - we just drop the updates of yml files or file chunks that aren't there.

If the cherry pick is more complicated, I'll create another PR.

@t8m t8m added branch: master Applies to master branch approval: review pending This pull request needs review by a committer branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL) approval: otc review pending triaged: bug The issue/pr is/fixes a bug branch: 3.0 Applies to openssl-3.0 branch branch: 3.1 Applies to openssl-3.1 (EOL) labels Nov 2, 2022
@t8m t8m mentioned this pull request Nov 2, 2022
@t8m t8m force-pushed the gh-actions-version-update branch from 3e320e0 to fed74cd Compare November 2, 2022 08:41
@t8m
Copy link
Copy Markdown
Member Author

t8m commented Nov 2, 2022

rebased to resolve conflict after moving the sctp daily job

Still OK?

@t8m
Copy link
Copy Markdown
Member Author

t8m commented Nov 2, 2022

@hlandau ping

@mattcaswell mattcaswell added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Nov 2, 2022
@openssl-machine
Copy link
Copy Markdown
Collaborator

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests. Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.

openssl-machine pushed a commit that referenced this pull request Nov 3, 2022
Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from #19581)
openssl-machine pushed a commit that referenced this pull request Nov 3, 2022
Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from #19581)

(cherry picked from commit 4ff6634)
openssl-machine pushed a commit that referenced this pull request Nov 3, 2022
Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from #19581)

(cherry picked from commit ec33ed7)
@t8m
Copy link
Copy Markdown
Member Author

t8m commented Nov 3, 2022

Merged to all branches after fixing conflicts as described above. Thank you for the reviews.

openssl-machine pushed a commit that referenced this pull request Nov 3, 2022
Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from #19581)

(cherry picked from commit ec33ed7)
@t8m t8m closed this Nov 3, 2022
beldmit pushed a commit to beldmit/openssl that referenced this pull request Dec 26, 2022
Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl#19581)
a-kromm-rogii pushed a commit to a-kromm-rogii/openssl that referenced this pull request Mar 14, 2025
Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl#19581)

(cherry picked from commit ec33ed7)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals branch: master Applies to master branch branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL) branch: 3.0 Applies to openssl-3.0 branch branch: 3.1 Applies to openssl-3.1 (EOL) triaged: bug The issue/pr is/fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants