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

CI with Github Actions to build nightlies archives #124

Merged
merged 25 commits into from
Apr 30, 2024

Conversation

yarik2720
Copy link
Contributor

Adding this workflow allows to download already compiled plugins with needed files in archive.
Suddenly this requires to change project path hierarchy

@yarik2720 yarik2720 changed the title CI with Github Actions to build nightlies archieves CI with Github Actions to build nightlies archives Apr 29, 2024
@Alienmario
Copy link
Collaborator

I appreciate the effort, but the project structure is intentional to aid development.
It should be the build script's job to organize the output archive.

@yarik2720
Copy link
Contributor Author

@Alienmario now switched to another builder, that accepts current file hierarchy

@Alienmario Alienmario self-assigned this Apr 29, 2024
@Alienmario
Copy link
Collaborator

Looking good, permission to experiment? (I'm a ci noob)

@yarik2720
Copy link
Contributor Author

yarik2720 commented Apr 30, 2024

@Alienmario it start automatically work once this is in your repository, even not in master branch. CI will make build (based by yml settings) on every commit push and pull requests in those branches, where CI files already persist.
You can see results of every commit on branch ci here: https://github.com/yarik2720/SourceCoop/actions

@Alienmario
Copy link
Collaborator

Yeah I'm aware, was more talking about the yml editing.
Anyway I'll merge it in now and see if I can expand on it after.
This will surely make many people happy, thank you.

@Alienmario Alienmario merged commit 3c6fa1a into ampreeT:master Apr 30, 2024
@yarik2720 yarik2720 deleted the ci branch May 1, 2024 07:39
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