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

Enable merge queue #108

Merged
merged 4 commits into from
May 24, 2023

Conversation

leandro-lucarella-frequenz
Copy link
Contributor

  • Test docs generation more widely
  • Be a bit more strict when limiting to tags
  • Trigger CI workflow on merge queues
  • Add a dummy workflow to succeed DCO check on merge queues

llucax added 4 commits May 24, 2023 14:04
We allow the docs generation test to run for any trigger but `push`, so
for exampmle `workflow_dispatch` will also run the test.  We rename the
job accordingly also to `test-generate-docs`.

The `push` event is still skipped because it already runs a job to
generate the final documentation, so we don't need to test it too.

Signed-off-by: Leandro Lucarella <[email protected]>
For this to work properly we need to exclude the branches starting with
`gh-readonly-queue/` for pushes, otherwise PRs will be tested twice,
once as the PR and the second one when they are pushed to the
`gh-readonly-queue/...` branch.

We do run the tests for the merge queue merge commit as it might include
several PRs being merged at the same time, so we want to test the merge.

Signed-off-by: Leandro Lucarella <[email protected]>
The DCO bot is not running on merge queues, so if we require the check
it will hang forever for merge queues.

See frequenz-floss/frequenz-sdk-python#363 for
more details.

Signed-off-by: Leandro Lucarella <[email protected]>
@leandro-lucarella-frequenz leandro-lucarella-frequenz requested a review from a team as a code owner May 24, 2023 12:09
@leandro-lucarella-frequenz leandro-lucarella-frequenz added this to the v0.16.0 milestone May 24, 2023
@leandro-lucarella-frequenz leandro-lucarella-frequenz added the type:tech-debt Improves the project without visible changes for users label May 24, 2023
@github-actions github-actions bot added the part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) label May 24, 2023
@leandro-lucarella-frequenz
Copy link
Contributor Author

Enabled auto-merge.

@leandro-lucarella-frequenz leandro-lucarella-frequenz added this pull request to the merge queue May 24, 2023
Merged via the queue into frequenz-floss:v0.x.x with commit 44e81a2 May 24, 2023
@leandro-lucarella-frequenz leandro-lucarella-frequenz deleted the merge-queue branch May 24, 2023 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) type:tech-debt Improves the project without visible changes for users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants