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

chore: update github actions #2520

Merged
merged 1 commit into from
Jun 20, 2023
Merged

chore: update github actions #2520

merged 1 commit into from
Jun 20, 2023

Conversation

galargh
Copy link
Contributor

@galargh galargh commented Jun 16, 2023

@galargh galargh marked this pull request as ready for review June 16, 2023 13:52
@galargh galargh requested review from a team, SgtPooki and whizzzkid as code owners June 16, 2023 13:52
Copy link
Contributor

@whizzzkid whizzzkid left a comment

Choose a reason for hiding this comment

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

This looks good, are any surprises expected?

continue-on-error: true # empty steps.tag.outputs.tag will inform the next step

- name: Build binaries with electron-builder
uses: samuelmeuli/action-electron-builder@92327c67bc45ff7c38bf55d8aa8c4d75b7ea38e7 # v1.6.0 but safer than a tag that can be changed
uses: paneron/action-electron-builder@14b133702d1b2e9749912051c43ed62b4afe56c8 # v1.8.1
Copy link
Contributor

Choose a reason for hiding this comment

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

can we remove the old actions from: https://github.com/ipfs/ipfs-desktop/settings/actions?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will do, after the merge.

@@ -133,7 +133,7 @@ jobs:
run: npm run test

- name: Test end-to-end
uses: GabrielBB/xvfb-action@86d97bde4a65fe9b290c0b3fb92c2c4ed0e5302d # v1.6
uses: coactions/setup-xvfb@b6b4fcfb9f5a895edadc3bc76318fae0ac17c8b3 # v1.0.1
Copy link
Contributor

Choose a reason for hiding this comment

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

this looks like a downgrade?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's a different action so the versions are not really comparable. The one used here before is not longer maintained 😢

@galargh
Copy link
Contributor Author

galargh commented Jun 19, 2023

This looks good, are any surprises expected?

Hopefully not, as far as I can tell the operations should continue as before.

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