Clean up CI: simplify dev versioning and remove dead config - #264
Merged
Conversation
- Replace VERSION.minor date-stamp machinery with a plain x.y.z.dev0 suffix for non-release builds (matches AutoGluon's scheme; no nightly publish here). - Remove the unused install_tabular/install_multimodal block from test_cloud.sh and the install_* / install_latest_* helpers from env_setup.sh; the DLC already ships AutoGluon, so the host only needs autogluon.cloud[tests]. - Drop the push trigger on the nonexistent dev branch. - Align configure-aws-credentials to @v5 in the composite action. - Remove tox / pytest-cov from test extras; drop pypandoc/packaging from the PyPI release build deps. - Delete unused [tool.mypy] and [tool.coverage.*] config (never invoked). The disabled (if: false) test_multimodal_cloud and test_cluster jobs are kept as placeholders for future re-enablement.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Housekeeping pass over CI/build config, removing things that unambiguously serve no purpose.
Versioning
VERSION.minordate-stamp machinery insetup.pywith a plainx.y.z.dev0suffix for non-release builds.RELEASE=1(tagged release) →0.5.1; otherwise →0.5.1.dev0. Matches AutoGluon's scheme, minus the nightly-counter logic we don't need (no nightly publish for autogluon.cloud). Deletes the tracked-but-gitignoredVERSION.minorfile and its.gitignoreentry.Dead install machinery
install_tabular/install_multimodalblock fromtest_cloud.shand theinstall_*/install_latest_*helpers fromenv_setup.sh. The DLC already ships AutoGluon, so the CI host only needsautogluon.cloud[tests](which pullsautogluon.common).Triggers / actions
pushtrigger on the nonexistentdevbranch (nowmaster,stable,v*tags).aws-actions/configure-aws-credentialsto@v5in the composite action (was@v1— inconsistent with the rest of the repo and on the deprecated Node16 runtime).Build/test config
toxandpytest-covfrom thetestsextra (never invoked).pypandoc/packagingfrom the PyPI release build deps (unused).[tool.mypy]and[tool.coverage.*]frompyproject.toml(never run).Kept intentionally
if: false)test_multimodal_cloudandtest_clusterjobs remain as placeholders for future re-enablement, along withtest_cluster.shand thepytest-forkedinstall its--forkedrun needs.Verification
pyproject.tomlparse.update_versionproduces0.5.1.dev0(unset) and0.5.1(RELEASE=1).setup.pypassesruff format/check; pre-commit hooks pass.