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 Linux Github Actions #75

Merged
merged 1 commit into from
Feb 9, 2025
Merged

Add Linux Github Actions #75

merged 1 commit into from
Feb 9, 2025

Conversation

nsauzede
Copy link
Contributor

@nsauzede nsauzede commented Feb 8, 2025

[EDITED TO ONLY CONTAIN GHA CHANGES]

Trivial PR to add a basic Linux (archlinux) Github Actions CI, in order to help track regressions.

I guess that an extra commit will be needed after merge into upstream master, to fix the GHA Badge URLS currently pointing to my fork, in README.md, ie:
Change:

[WorkflowBadgeLinux]: https://github.com/nsauzede/darkriscv/workflows/Linux/badge.svg
[WorkflowUrlLinux]: https://github.com/nsauzede/darkriscv/actions/workflows/Linux.yml

Into:

[WorkflowBadgeLinux]: https://github.com/darklife/darkriscv/workflows/Linux/badge.svg
[WorkflowUrlLinux]: https://github.com/darklife/darkriscv/actions/workflows/Linux.yml

@samsoniuk samsoniuk self-assigned this Feb 9, 2025
@samsoniuk
Copy link
Member

wow! thanks again! I would ask you to submit the chances on separate PRs, because I am really not much sure about remove the trailing spaces and I need more time to think about it...

@nsauzede nsauzede changed the title Add Linux Github Actions + remove useless 'x' perm and trailing spaces Add Linux Github Actions Feb 9, 2025
@nsauzede
Copy link
Contributor Author

nsauzede commented Feb 9, 2025

Sure thing! Sorry I really should not have done the two tasks at once.
I trimmed this PR to only add GHA.
I will propose the cleanup in separate PRs.

@nsauzede
Copy link
Contributor Author

nsauzede commented Feb 9, 2025

Done:
#77
#78

@samsoniuk samsoniuk merged commit 118f705 into darklife:master Feb 9, 2025
@samsoniuk
Copy link
Member

thanks again! :)

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