Skip to content

ci: add vpm workflow - #20177

Merged
spytheman merged 5 commits into
vlang:masterfrom
ttytm:ci/vpm
Dec 15, 2023
Merged

spytheman merged 5 commits into
vlang:masterfrom
ttytm:ci/vpm

Conversation

@ttytm

@ttytm ttytm commented Dec 14, 2023 •

Copy link
Copy Markdown
Member

Due to the recent addition of the requirement to add -d network to include VPM tests into test runs, there is no CI coverage for VPM. The tests are always skipped because there are no runners defining this flag.

This PR adds a VPM workflow. It only runs when VPM related files or the workflow file have been changed, so there is no additional overhead when other changes are made in the codebase.

https://github.com/ttytm/v/actions/runs/7212169683

🤖[deprecated] Generated by Copilot at 35077c7

Add a GitHub Actions workflow to run CI tests for vpm on different OS and C compilers. The workflow file is .github/workflows/vpm_ci.yml.

🤖[deprecated] Generated by Copilot at 35077c7

  • Add a new workflow file for GitHub Actions to run CI tests for vpm (F0)

@spytheman

Copy link
Copy Markdown
Contributor

Thanks, I was going to add them eventually to .github/workflows/periodic_ci.yml, but a new one is even better.

Comment thread .github/workflows/vpm_ci.yml Outdated
Comment thread .github/workflows/vpm_ci.yml Outdated
Co-authored-by: Delyan Angelov <26967+spytheman@users.noreply.github.com>
@ttytm
ttytm marked this pull request as draft December 14, 2023 21:07
@ttytm
ttytm marked this pull request as ready for review December 14, 2023 21:17
@spytheman

Copy link
Copy Markdown
Contributor

Excellent work.

@spytheman
spytheman merged commit c5a6aeb into vlang:master Dec 15, 2023
@ttytm
ttytm deleted the ci/vpm branch December 15, 2023 22:06
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