dabla commented on PR #73708: URL: https://github.com/apache/airflow/pull/73708#issuecomment-5829211579
Yes. Nothing about the code has changed since the two providers were merged, and both are shippable in their current state. **IBM MQ.** `git diff --stat 23b334f29eb..main -- providers/ibm/mq/src providers/ibm/mq/tests` shows three touched files, none behavioural: the namespace `__init__.py` lost a `# type: ignore` when Db2 joined the namespace (#67532), #73588 moved the queue provider's docstring block into `docs/message-queues.rst`, and one test assertion was adjusted for the `queue_uri` rename in common.messaging (#71346). Hooks and triggers are byte-for-byte what was merged in May. The remaining commits (#67757, #72058) only added the ARM `excluded-platforms` entries in `provider.yaml`. This is also the exact code we have been running in production at Infrabel on Airflow 3.3.x since spring for event-driven scheduling on IBM MQ. **IBM Db2.** Since #67532 the only code change is #72025, which skips `None`-valued extras when building the connection string, plus its tests; the rest is a docs spellcheck fix (#72203) and the ARM exclusion (#72058). What I checked on this branch just now: * `uv build` produces sdist and wheel for both providers, and `twine check` passes on all four artifacts. * Db2 unit tests: 23 passed. IBM MQ unit tests: 93 passed; the 12 remaining tests errored at collection in my local environment because it lacks the `airflow_shared` devel package, not because of the provider. They run in this PR's CI, which already exercises `Providers[ibm.mq]` and `Providers[ibm.db2]` on every main build since not-ready providers are tested like regular ones. * Both changelogs already carry the 0.1.0 "Initial version" entry, so `prepare-provider-documentation` needs no classification. One operational note, unrelated to the packages themselves: the lowest-dependency CI job for `ibm.mq` downloads the MQ C client from IBM's server and has flaked on that download; #71544 adds a timeout and retries. It does not affect the released distribution, which depends on the `ibmmq` binding only through an optional extra. --- Drafted-by: Claude Code (Fable 5.1) (no human review before posting) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
