Skip to content

Redesign Github workflows#59

Open
FooBarWidget wants to merge 6 commits into
mainfrom
pr_ci
Open

Redesign Github workflows#59
FooBarWidget wants to merge 6 commits into
mainfrom
pr_ci

Conversation

@FooBarWidget
Copy link
Copy Markdown
Member

@FooBarWidget FooBarWidget commented May 13, 2026

  • Have a single test-and-build workflow for testing everything as well as building the API server. Get rid of the separate "build API server" workflow.
  • Have the deploy workflow trigger based on successful completion of API server build on the main branch.
  • Syntax check Ansible.
  • Security audit workflows using zizmor — thanks TanStack for the tip.
  • Make test-and-build workflow compatible with pull requests.
  • Enforce timeouts.

- Have a single test workflow for testing everything, including the API server.
- Split the API server workflow into a build part and a deploy part. Get rid of test steps, since that's done by the test workflow.
- Syntax check Ansible.
- Make test and API server build workflows compatible with pull requests.
- Enforce timeouts.
@FooBarWidget FooBarWidget requested review from abtreece and rus-max May 13, 2026 16:01
@github-advanced-security
Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

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