Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cleaning unnecessary files from repo root #17235

Merged
merged 6 commits into from
Feb 21, 2025
Merged

cleaning unnecessary files from repo root #17235

merged 6 commits into from
Feb 21, 2025

Conversation

zzstoatzz
Copy link
Collaborator

@zzstoatzz zzstoatzz commented Feb 21, 2025

removes:

  • .ruff.toml was completely redundant bc the config was already in the pyproject.toml
  • .codespellrc config into there as well and removes the extra file
  • settings_schema.json (which has not changed in months) that lived at repo root since I assume its been superseded by schema/ (which has recently changed) but cc @desertaxle
  • .gitattributes which was referring to src/prefect/_version which no longer exists with the new pyproject setup

@zzstoatzz zzstoatzz added the development Tech debt, refactors, CI, tests, and other related work. label Feb 21, 2025
@zzstoatzz zzstoatzz changed the title rm redundant .ruff.toml file more repo root cleaning Feb 21, 2025
@zzstoatzz zzstoatzz changed the title more repo root cleaning cleaning unnecessary files from repo root Feb 21, 2025
Copy link
Member

@cicdw cicdw left a comment

Choose a reason for hiding this comment

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

🧹

Copy link

codspeed-hq bot commented Feb 21, 2025

CodSpeed Performance Report

Merging #17235 will not alter performance

Comparing remove-rufftoml (fb92540) with main (29748d5)

Summary

✅ 2 untouched benchmarks

Copy link
Member

Choose a reason for hiding this comment

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

Yup, this was definitely committed in error

@zzstoatzz zzstoatzz merged commit a7e94ad into main Feb 21, 2025
47 checks passed
@zzstoatzz zzstoatzz deleted the remove-rufftoml branch February 21, 2025 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
development Tech debt, refactors, CI, tests, and other related work.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants