Skip to content

Big cleanup#126

Merged
javfg merged 6 commits intodevfrom
big-cleanup
Apr 24, 2025
Merged

Big cleanup#126
javfg merged 6 commits intodevfrom
big-cleanup

Conversation

@javfg
Copy link
Member

@javfg javfg commented Apr 15, 2025

Big list of cleanups:

  • Switch to uv, as airflow finally fixed the bug that was breaking installation.
  • Use python 3.12 instead of 3.10.
  • Rename package from ot_orchestration to orchestration.
  • Add lint rules and lint files with them.
  • Add vscode settings and extension recommendations.
  • Clean yaml files, gitignore, dockerfile, compose file, makefile.
  • Refactored PR github action to use uv.
  • Removed beautysh from pre-commit, it seems abandoned and works badly without setuptools.

This PR should also be rebased on top of dev, without squashing the commits (and without a merge commit either!).

@javfg javfg requested a review from project-defiant April 15, 2025 10:47
Copy link
Collaborator

@project-defiant project-defiant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see nothing harmful :) Thank you for the clean up and uv

@javfg javfg merged commit 12603a0 into dev Apr 24, 2025
2 checks passed
@javfg javfg deleted the big-cleanup branch April 24, 2025 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

uv fails to resolve apache-airflow[all]==2.8.4 where pip succeeds

2 participants