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

GitHub actions #5

Merged
merged 5 commits into from
Mar 26, 2023
Merged

GitHub actions #5

merged 5 commits into from
Mar 26, 2023

Conversation

Antonov548
Copy link
Contributor

@Antonov548 Antonov548 commented Mar 15, 2023

For #1.

Copy link
Contributor

@krlmlr krlmlr left a comment

Choose a reason for hiding this comment

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

The action is currently failing because a file that always exists on GitPod doesn't exist here. Can we work around?

@krlmlr
Copy link
Contributor

krlmlr commented Mar 15, 2023

@Antonov548
Copy link
Contributor Author

@krlmlr up and running 👍

Copy link
Contributor

@krlmlr krlmlr left a comment

Choose a reason for hiding this comment

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

Nice, thanks!

- master

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}-${{ github.head_ref || '' }}-${{ github.base_ref || '' }}
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm using a slightly different setting in my repos now, to make sure that we build all commits on the main branch. Can be changed in a forthcoming PR.

@krlmlr krlmlr merged commit 70b6548 into main Mar 26, 2023
@krlmlr krlmlr deleted the feat/ci branch March 26, 2023 04:59
@krlmlr
Copy link
Contributor

krlmlr commented Mar 26, 2023

Can you please proceed with the next point in #1 -- create a static build matrix to test all projects?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants