Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump GitHub actions to latest versions #1551

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

deining
Copy link
Contributor

@deining deining commented Aug 14, 2023

This PR bumps GitHub actions to their latest versions. It also fixes a typo in a comment on the way.

@tdelmas
Copy link
Collaborator

tdelmas commented Aug 14, 2023

I think it's better to update HUGO_VERSION in a separate PR

@deining
Copy link
Contributor Author

deining commented Aug 15, 2023

I think it's better to update HUGO_VERSION in a separate PR

I reworked the PR, it now containes updates for the actions only.

If wanted, I can author a PR bumping the HUGO_VERSION. What version do hugo do you want? Is it fine to use the latest version v0.117.0?

@deining
Copy link
Contributor Author

deining commented Aug 15, 2023

I think it's better to update HUGO_VERSION in a separate PR

I reworked the PR, it now containes updates for the actions only.

If wanted, I can author another PR bumping the HUGO_VERSION. What version do hugo do you want? Is it fine to use the latest version v0.117.0?

@tdelmas tdelmas requested a review from scottmakestech August 15, 2023 11:14
@tdelmas
Copy link
Collaborator

tdelmas commented Aug 15, 2023

If wanted, I can author another PR bumping the HUGO_VERSION. What version do hugo do you want? Is it fine to use the latest version v0.117.0?

Sure! The latest version if possible. If I remember correctly, it needs to be updated in two places. Here are the instructions to do it:

https://github.com/letsencrypt/website/blob/main/DOCUMENTATION.md#how-to-upgrade-hugo
(The step "3. Check the diff" must be checked carefully to avoid unwanted surprises.)

@deining
Copy link
Contributor Author

deining commented Sep 29, 2023

Friendly ping.
@scottmakestech: Any objections against merging my PR?

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.

2 participants