potiuk commented on PR #49829: URL: https://github.com/apache/airflow/pull/49829#issuecomment-2833411921
OK. maybe not as simple as the original change, but I think better overall: * we only have single flag `--version-suffix` now (instead of `--version-for-pypi/--version-forlocal`) * it handles all the weird s**t for versions: rc1, .dev0, .post1, but also `rc1.dev0` , `a2.post1`, `rc1+local1`, `rc1.dev0.local1` (ll of those are valid suffixes) * unified the code - both provider and non-provider version update is now consistently named and both use context manager to restore original version * all that is automatically tested with unit tests * code is moved to the modules it belongs to -- 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
