The GitHub Actions job "Tests" on airflow.git has failed.
Run started by GitHub user potiuk (triggered by jedcunningham).

Head commit for run:
060eeb73d3636f7ff692acb6c61c7ff948c3574f / Daniel Standish 
<15932138+dstand...@users.noreply.github.com>
Rename latest_version to version in bundle orm / code clarification (#45720)

1. did a little work to try to make the code a little clearer in the dag 
processor where we compare versions.  Instead of current and new, which is 
confusing, I call it pre-refresh and after-refresh, essentially, which makes 
the logic a little more intelligible.  Also I provide a var `was seen` to show 
the intention of the `name in list` check.

2. I propose (and do so here) renaming `latest_version` to `version` in bundle 
for reasons similar to https://github.com/apache/airflow/pull/45719.  I think 
it makes sense to think of the orm object as _itself_ representing the latest 
or current or last refreshed version. So the latest version cannot itself 
_have_ a latest version -- the latest version just has a version.  That's sorta 
conceptually the issue.  In any event, by saying less, we can let the docs 
explain the nuance.

---------

Co-authored-by: Jed Cunningham <66968678+jedcunning...@users.noreply.github.com>

Report URL: https://github.com/apache/airflow/actions/runs/12821136614

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: jobs-unsubscr...@airflow.apache.org
For additional commands, e-mail: jobs-h...@airflow.apache.org

Reply via email to