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

Add gh action to upload artifacts #398

Merged
merged 8 commits into from
Feb 12, 2024

Conversation

selankon
Copy link
Collaborator

@selankon selankon commented Feb 9, 2024

FTR

Limeapp automatic preview builds.

Now when a PR is marked with the Labels "artifact", an additional GH action will run to upload a built version of the limeapp on GH.

This artifact will store the PR number, to on posterior publish a GH comment on the PR with the artifacts url.

Example:

#398 (comment)

The github action to publish the comment runs on the develop branch and not on the pr to protect from external forks PR's the secrets and other configurations (to comment on the PR the action needs write and read permissions, which are dangerous to give to a fork from an external repository).

Further reading

Keeping your GitHub Actions and workflows secure

https://securitylab.github.com/research/github-actions-preventing-pwn-requests/

https://glasnt.com/blog/pull_request_target_labels/

Workflow run docs

https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#workflow_run
https://docs.github.com/en/webhooks/webhook-events-and-payloads#workflow_run

@selankon selankon added the artifact Will triger a GH action to create an artifact label Feb 9, 2024
@selankon selankon force-pushed the f/upload-artifacts branch 2 times, most recently from 94e7553 to bcaa121 Compare February 9, 2024 17:50
@selankon selankon added artifact Will triger a GH action to create an artifact and removed artifact Will triger a GH action to create an artifact labels Feb 9, 2024
@selankon selankon added artifact Will triger a GH action to create an artifact and removed artifact Will triger a GH action to create an artifact labels Feb 12, 2024
@selankon selankon added artifact Will triger a GH action to create an artifact and removed artifact Will triger a GH action to create an artifact labels Feb 12, 2024
@selankon selankon force-pushed the develop branch 12 times, most recently from e11390e to c749df1 Compare February 12, 2024 17:42
Copy link

@G10h4ck G10h4ck merged commit b292539 into libremesh:develop Feb 12, 2024
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
artifact Will triger a GH action to create an artifact
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants