Skip to content

Conversation

@ErwanSimonetti
Copy link
Contributor

@ErwanSimonetti ErwanSimonetti commented Mar 7, 2025

Currently, we have to trigger a workflow manually to release sql-compare on PyPI.

We should trigger this workflow automatically whenever we create a release on sql-compare, like we do for pytest-mergify.

Fixes MRGFY-4860

@mergify
Copy link
Contributor

mergify bot commented Mar 7, 2025

Merge Protections

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

🟢 Changelog requirements

Wonderful, this rule succeeded.
  • any of:
    • label = skip changelog
    • -title ~= ^feat
    • label = need 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 March 7, 2025 10:29
@ErwanSimonetti ErwanSimonetti force-pushed the automatic-pypi-release branch from fa0e7d2 to 443346f Compare March 7, 2025 10:51
@mergify mergify bot dismissed DouglasBlackwood’s stale review March 7, 2025 10:51

Pull request has been modified.

@mergify mergify bot requested a review from a team March 7, 2025 10:52
@Syffe
Copy link

Syffe commented Mar 7, 2025

Usually, it's a good practice to add some description of the commit in the body of the pull request, to quickly have a context of what's going on !

@ErwanSimonetti
Copy link
Contributor Author

Usually, it's a good practice to add some description of the commit in the body of the pull request, to quickly have a context of what's going on !

Got it, will do next time 🙂

@mergify mergify bot merged commit f201dfe into Mergifyio:main Mar 7, 2025
8 checks passed
@mergify mergify bot added queued and removed queued labels Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants