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

49 fix build #50

Merged
merged 3 commits into from
Oct 2, 2024
Merged

49 fix build #50

merged 3 commits into from
Oct 2, 2024

Conversation

Phu2
Copy link
Contributor

@Phu2 Phu2 commented Sep 30, 2024

Even though ubuntu-latest or ubuntu-24.04 is avalaible https://github.com/actions/runner-images?tab=readme-ov-file#available-images the build error says

The current runner (ubuntu-24.04-x64) was detected as self-hosted because the platform does not match a GitHub-hosted runner image (or that image is deprecated and no longer supported).

That's why i am resetting to the predesscor version ubuntu-22.04

@Phu2 Phu2 requested a review from TobiasNx September 30, 2024 09:45
@Phu2
Copy link
Contributor Author

Phu2 commented Sep 30, 2024

@TobiasNx Build now runs successfully, deployment fails only because of the wrong branch. After merging into master it will deploy with success i think.

@Phu2
Copy link
Contributor Author

Phu2 commented Sep 30, 2024

Well, as you can see here https://github.com/organizations/metafacture/settings/actions/hosted-runners ubuntu-24.04 is not available for this project at the moment.

@Phu2
Copy link
Contributor Author

Phu2 commented Sep 30, 2024

Wait, but ubuntu-latest is available 🤔

@Phu2
Copy link
Contributor Author

Phu2 commented Sep 30, 2024

Anyway, please approve and merge, @TobiasNx

@TobiasNx
Copy link
Collaborator

TobiasNx commented Oct 2, 2024

+1

@TobiasNx TobiasNx merged commit 93e4d36 into master Oct 2, 2024
1 of 2 checks passed
@TobiasNx TobiasNx deleted the 49-fix-build branch October 2, 2024 09:28
@TobiasNx TobiasNx mentioned this pull request Oct 9, 2024
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