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.

Sign up for GitHub

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

Jump to bottom

chore: update CI triggers to minify useless runs #1483

Merged
merged 27 commits into from
Jun 3, 2024
Merged

chore: update CI triggers to minify useless runs #1483

merged 27 commits into from
Jun 3, 2024

Conversation

Lancetnik
Copy link
Member

No description provided.

Lancetnik and others added 18 commits May 29, 2024 17:12
@Lancetnik
chore: run CQ CI only with PR and by cron
38d1c51
@Lancetnik
chore: run DR CI only if pyproject changed
1c49bd5
@Lancetnik
format: correct deploy docs CI
c78bcc4
@Lancetnik
chore: run coverage publish CI only after PR merged
0c35e2c
@Lancetnik
chore: run tests only in PR
dadc8ae
@Lancetnik
chore: fix test CI
0ea7a6b
@Lancetnik
chore: add CI workflows names conversation
f88ce8e
@Lancetnik
chore: add docs update references CI
6a7b82b
@Lancetnik
chore: make update references CI writeable
f9565a2
@Lancetnik
chore: finalize CI
54c9bee
@Lancetnik
chore: correct name in check links CI
0b6f657
@Lancetnik
chore: run tests only if required
4529728
@Lancetnik
chore: run CQ CI if required
b5e2aae
@Lancetnik
chore: fix broken links CI
47c08e6
@Lancetnik
chore: make References autocommit
e186329
@Lancetnik @github-actions
Apply automatic changes
81da016
@Lancetnik
chore: change GHA commit message
e5dee88
@Lancetnik
Merge branch 'chore/ci-triggers' of github.com:airtai/faststream into…
1b835f1 
… chore/ci-triggers
@Lancetnik Lancetnik requested review from kumaranvpl, davorrunje and sternakt and removed request for kumaranvpl, davorrunje and sternakt May 30, 2024 18:22
kumaranvpl
kumaranvpl previously approved these changes May 31, 2024
View reviewed changes
@kumaranvpl kumaranvpl enabled auto-merge June 3, 2024 05:50
@kumaranvpl kumaranvpl added this pull request to the merge queue Jun 3, 2024
@Lancetnik Lancetnik removed this pull request from the merge queue due to a manual request Jun 3, 2024
@Lancetnik
Copy link
Member Author

Seems like I found the problem: https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/configuring-pull-request-merges/managing-a-merge-queue#triggering-merge-group-checks-with-github-actions

@Lancetnik Lancetnik requested a review from kumaranvpl June 3, 2024 06:12
kumaranvpl
kumaranvpl previously approved these changes Jun 3, 2024
View reviewed changes
@kumaranvpl kumaranvpl enabled auto-merge June 3, 2024 06:19
@Lancetnik Lancetnik linked an issue Jun 3, 2024 that may be closed by this pull request
Fix CI #1494
Closed
@Lancetnik Lancetnik disabled auto-merge June 3, 2024 06:28
@Lancetnik
Copy link
Member Author

@kumaranvpl sorry, it was the last change :)

kumaranvpl
kumaranvpl previously approved these changes Jun 3, 2024
View reviewed changes
@kumaranvpl kumaranvpl enabled auto-merge June 3, 2024 06:37
@kumaranvpl kumaranvpl added this pull request to the merge queue Jun 3, 2024
@Lancetnik Lancetnik removed this pull request from the merge queue due to a manual request Jun 3, 2024
@Lancetnik
Copy link
Member Author

I am still not sure, why it doesn't run check in MG

@kumaranvpl kumaranvpl enabled auto-merge June 3, 2024 07:14
@kumaranvpl kumaranvpl added this pull request to the merge queue Jun 3, 2024
Merged via the queue into main with commit b8b003d Jun 3, 2024
29 checks passed
@kumaranvpl kumaranvpl deleted the chore/ci-triggers branch June 3, 2024 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kumaranvpl kumaranvpl kumaranvpl approved these changes

@davorrunje davorrunje Awaiting requested review from davorrunje

@sternakt sternakt Awaiting requested review from sternakt

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fix CI
2 participants
@Lancetnik @kumaranvpl