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

ci: add yamllint #4

Merged

Conversation

jd
Copy link
Member

@jd jd commented Dec 11, 2024

Depends-On: #1

@jd jd mentioned this pull request Dec 11, 2024
@jd
Copy link
Member Author

jd commented Dec 11, 2024

This pull request is part of a stack:

  1. chore: remove print (#1)
  2. ci: add yamllint (#4) 👈
  3. chore: set version to 0 (#6)

@jd jd mentioned this pull request Dec 11, 2024
@jd jd force-pushed the devs/jd/remove-print/I8d30c11daf3df0067fff3cf404e639d2b4b63d47 branch from 0604b72 to 30dcfb3 Compare December 11, 2024 12:54
@jd jd force-pushed the devs/jd/remove-print/Ia9eb6ded1a73fb3d428d06719129de58e0ed50ef branch from bb0335b to 758014c Compare December 11, 2024 12:54
This was referenced Dec 11, 2024
@jd jd force-pushed the devs/jd/remove-print/I8d30c11daf3df0067fff3cf404e639d2b4b63d47 branch from 30dcfb3 to 09bbe7b Compare December 11, 2024 12:57
@jd jd force-pushed the devs/jd/remove-print/Ia9eb6ded1a73fb3d428d06719129de58e0ed50ef branch from 758014c to 3fa8381 Compare December 11, 2024 12:57
@jd jd force-pushed the devs/jd/remove-print/I8d30c11daf3df0067fff3cf404e639d2b4b63d47 branch from 09bbe7b to 3d0a0c8 Compare December 11, 2024 13:15
@jd jd force-pushed the devs/jd/remove-print/Ia9eb6ded1a73fb3d428d06719129de58e0ed50ef branch from 3fa8381 to ff6819c Compare December 11, 2024 13:15
jd added 2 commits December 11, 2024 14:43
Change-Id: I8d30c11daf3df0067fff3cf404e639d2b4b63d47
Change-Id: Ia9eb6ded1a73fb3d428d06719129de58e0ed50ef
@jd jd force-pushed the devs/jd/remove-print/I8d30c11daf3df0067fff3cf404e639d2b4b63d47 branch from 3d0a0c8 to 2660aae Compare December 11, 2024 13:43
@jd jd force-pushed the devs/jd/remove-print/Ia9eb6ded1a73fb3d428d06719129de58e0ed50ef branch from ff6819c to ce9ab10 Compare December 11, 2024 13:43
Copy link

mergify bot commented Dec 11, 2024

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 ⛓️ Depends-On Requirements

Wonderful, this rule succeeded.

Requirement based on the presence of Depends-On in the body of the pull request

🟢 Reviews

Wonderful, this rule succeeded.
  • #approved-reviews-by >= 1

🟢 Continuous Integration

Wonderful, this rule succeeded.
  • all of:
    • check-success = test (3.10)
    • check-success = test (3.11)
    • check-success = test (3.12)
    • check-success = test (3.13)

🟢 No review blocker

Wonderful, this rule succeeded.
  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

🟢 Changelog requirements

Wonderful, this rule succeeded.
  • any of:
    • -title ~= ^feat
    • label = need changelog
    • label = skip changelog

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?:

🟢 🔎 Reviews

Wonderful, this rule succeeded.
  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

@mergify mergify bot requested a review from a team December 11, 2024 13:45
Base automatically changed from devs/jd/remove-print/I8d30c11daf3df0067fff3cf404e639d2b4b63d47 to main December 11, 2024 13:47
@mergify mergify bot added the queued label Dec 11, 2024
@mergify mergify bot merged commit df4cb25 into main Dec 11, 2024
9 checks passed
@mergify mergify bot deleted the devs/jd/remove-print/Ia9eb6ded1a73fb3d428d06719129de58e0ed50ef branch December 11, 2024 13:50
@mergify mergify bot removed the queued label Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants