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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: fix v-analyzer workflow #20178

Merged
merged 1 commit into from Dec 14, 2023
Merged

ci: fix v-analyzer workflow #20178

merged 1 commit into from Dec 14, 2023

Conversation

ttytm
Copy link
Member

@ttytm ttytm commented Dec 14, 2023

Related to the recent changes over at v-analyzer using a submodule for treesitter.

馃[deprecated] Generated by Copilot at f377cc5

Modified the git clone command in .github/workflows/v_apps_and_modules_compile_ci.yml to use shallow submodules for v-analyzer. This can improve the workflow speed and disk space usage.

馃[deprecated] Generated by Copilot at f377cc5

  • Add a new workflow for v-analyzer to run on every push and pull request (-)
  • Clone the v-analyzer repository and its submodules with minimal depth and size using --recursive and --shallow-submodules flags (link)

@spytheman spytheman merged commit 22cb9c5 into vlang:master Dec 14, 2023
24 checks passed
@ttytm ttytm deleted the ci/fix-v-analyzer branch December 14, 2023 18:11
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.

None yet

2 participants