Skip to content
This repository was archived by the owner on Jan 9, 2025. It is now read-only.

Update to newest version of github actions for python and artifact #8

Open
evelynmitchell wants to merge 1 commit intofacebook:mainfrom
evelynmitchell:main
Open

Update to newest version of github actions for python and artifact #8
evelynmitchell wants to merge 1 commit intofacebook:mainfrom
evelynmitchell:main

Conversation

@evelynmitchell
Copy link
Copy Markdown

python and artifact-upload

I've done the CLA.

The python action and the artifact-upload action versions that were being used were deprecated. This PR updates them to the latest version.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 17, 2024
@arthaud
Copy link
Copy Markdown

arthaud commented Jan 2, 2025

Thanks for doing this.

I see more errors in the logs: Error: This request has been automatically failed because it uses a deprecated version of actions/download-artifact: v2
https://github.com/facebook/pysa-action/actions/runs/12379570623/job/35062910516?pr=8

I think we need to upgrade download-artifact too. Could you update that as well?

@evelynmitchell
Copy link
Copy Markdown
Author

Thanks for doing this.

I see more errors in the logs: Error: This request has been automatically failed because it uses a deprecated version of actions/download-artifact: v2 https://github.com/facebook/pysa-action/actions/runs/12379570623/job/35062910516?pr=8

I think we need to upgrade download-artifact too. Could you update that as well?

That error is coming out of an archived repository:

https://github.com/facebookarchive/sapp-action

Perhaps the correct path forward is to archive this repository?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants