Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Apr 5, 2023

The typo causes unnecessary delays on building regular PRs :( It was introduced in #30424


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

The typo causes unnecessary delays on building regular PRs :(
It was introduced in apache#30424
@potiuk potiuk merged commit 7aca81c into apache:main Apr 5, 2023
@potiuk potiuk deleted the fix-typo-in-build-info-output branch April 5, 2023 08:49
potiuk added a commit to potiuk/airflow that referenced this pull request Apr 5, 2023
(Facepalm) The typo of output -> outputs from apache#30482  was also in the
ci.yaml where it was used and it was missed in this PR.

I can blame the GitHub Actions stupid choice of accepting typoed
names of outputs and replacing them with blank strings (which I
raised as an issue a long time ago)
potiuk added a commit that referenced this pull request Apr 5, 2023
(Facepalm) The typo of output -> outputs from #30482  was also in the
ci.yaml where it was used and it was missed in this PR.

I can blame the GitHub Actions stupid choice of accepting typoed
names of outputs and replacing them with blank strings (which I
raised as an issue a long time ago)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants