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

Fix pr workflow #114

Merged
merged 1 commit into from
Nov 21, 2024
Merged

Fix pr workflow #114

merged 1 commit into from
Nov 21, 2024

Conversation

iofq
Copy link
Contributor

@iofq iofq commented Nov 19, 2024

Fixes #94

Does not run the deploy step unless for master branch. Also updated actions to latest versions.

@iofq iofq self-assigned this Nov 19, 2024
@iofq iofq requested a review from neolynx November 19, 2024 05:54
@iofq
Copy link
Contributor Author

iofq commented Nov 19, 2024

CI will fail until #113 is merged first

@neolynx
Copy link
Member

neolynx commented Nov 19, 2024

there was some automation when pushing to master, the feature/gh-action-pages branch would be updated...

how would updating the page work after this change ?

@iofq
Copy link
Contributor Author

iofq commented Nov 19, 2024

there was some automation when pushing to master, the feature/gh-action-pages branch would be updated...

how would updating the page work after this change ?

master behavior should be unchanged, this is to fix the deploy step from running on every pull request (even though it fails on pull requests anyway)

@neolynx neolynx merged commit f04753b into aptly-dev:master Nov 21, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Don't try to deploy pull requests
2 participants