pankajkoti commented on code in PR #36143:
URL: https://github.com/apache/airflow/pull/36143#discussion_r1421501109
##########
.github/workflows/ci.yml:
##########
@@ -762,14 +762,20 @@ jobs:
run: >
breeze release-management generate-issue-content-providers
--only-available-in-dist --disable-progress
+ - name: "Generate source constraints from CI image"
+ shell: bash
+ run: >
+ breeze release-management generate-constraints
+ --airflow-constraints-mode constraints-source-providers
- name: "Install and verify all provider packages and airflow via wheel
files"
run: >
breeze release-management verify-provider-packages
--use-packages-from-dist
--package-format wheel
--use-airflow-version wheel
--airflow-constraints-reference default
- --providers-constraints-reference constraints-main
+ --providers-constraints-location
+
/files/constraints-"${{env.PYTHON_MAJOR_MINOR_VERSION}}"/constraints-source-providers-${{env.PYTHON_MAJOR_MINOR_VERSION}}.txt
Review Comment:
```suggestion
/files/constraints-${{env.PYTHON_MAJOR_MINOR_VERSION}}/constraints-source-providers-${{env.PYTHON_MAJOR_MINOR_VERSION}}.txt
```
Not sure if we need the quotes there? I am just thinking that we did not
use quotes for the later reference
`constraints-source-providers-${{env.PYTHON_MAJOR_MINOR_VERSION}}.txt` for the
.txt file, so guessing we may not need in the earlier reference too? 🤔
--
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: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
