You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/06/15 22:35:06 UTC
[GitHub] [airflow] potiuk opened a new pull request #16470: Checks-out missing commits on selective checks
potiuk opened a new pull request #16470:
URL: https://github.com/apache/airflow/pull/16470
Recent improvement of the build images to use pull_request_target
removed by accident fetching of the PR commits which makes it
impossible to determine which files has changed in PR in the
PR build image workflow.
This commit brings it back.
<!--
Thank you for contributing! Please make sure that your code changes
are covered with tests. And in case of new features or big changes
remember to adjust the documentation.
Feel free to ping committers for the review!
In case of existing issue, reference it using one of the following:
closes: #ISSUE
related: #ISSUE
How to write a good git commit message:
http://chris.beams.io/posts/git-commit/
-->
---
**^ Add meaningful description above**
Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)** for more information.
In case of fundamental code change, Airflow Improvement Proposal ([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvements+Proposals)) is needed.
In case of a new dependency, check compliance with the [ASF 3rd Party License Policy](https://www.apache.org/legal/resolved.html#category-x).
In case of backwards incompatible changes please leave a note in [UPDATING.md](https://github.com/apache/airflow/blob/main/UPDATING.md).
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on pull request #16470: Checks-out missing commits on selective checks
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #16470:
URL: https://github.com/apache/airflow/pull/16470#issuecomment-861880623
This one failed https://github.com/apache/airflow/pull/16469 because the PR commits were missing https://github.com/apache/airflow/runs/2834014923?check_suite_focus=true#step:4:584
```
Changed files from 207fd82d2f90ba1b3d6d0e17820e5b15ef820f53 vs it's first parent
fatal: ambiguous argument '207fd82d2f90ba1b3d6d0e17820e5b15ef820f53^': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'
```
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org