Skip to content

Try a native aarch64 build - #167

Merged
Cadair merged 5 commits into
mainfrom
Cadair-patch-1
Nov 26, 2024
Merged

Cadair merged 5 commits into
mainfrom
Cadair-patch-1

Conversation

@Cadair

@Cadair Cadair commented Sep 15, 2024

Copy link
Copy Markdown
Member

for science

@Cadair
Cadair marked this pull request as draft September 15, 2024 15:18
Comment thread .github/workflows/ci.yml
test:
needs: [core, sdist_verify, rust]
uses: OpenAstronomy/github-actions-workflows/.github/workflows/tox.yml@main
uses: sunpy/github-actions-workflows/.github/workflows/tox.yml@main

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Apparently GH wont let you run cross-organization reuseable workflows on self-hosted or paid-for runners.

@Cadair

Cadair commented Sep 15, 2024

Copy link
Copy Markdown
Member Author

Before this gets there's a few things that need sorting, 1) a fork of the workflows (apparently) 2) the publish workflow needs to add aarch64 support 3) I need to be ok with paying for this.

@Cadair

Cadair commented Sep 17, 2024

Copy link
Copy Markdown
Member Author

I have made it so the sunpy fork of the workflows pulls weekly on a cron, so it shouldn't drift out of date. I now just need to make the publish workflows work.

@Cadair Cadair added the Run publish Run publish CI on this PR. label Nov 12, 2024
@Cadair

Cadair commented Nov 12, 2024

Copy link
Copy Markdown
Member Author

This works when using OpenAstronomy/github-actions-workflows#247, the only real downside is the need to use the workflows from within the org.

@Cadair
Cadair marked this pull request as ready for review November 26, 2024 11:28
Comment thread .github/workflows/ci.yml Resolved
@Cadair
Cadair merged commit 4a39f97 into main Nov 26, 2024
@Cadair
Cadair deleted the Cadair-patch-1 branch November 26, 2024 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Run publish Run publish CI on this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants