Skip to content

chore: simplify Haystack Hatch scripts#9491

Merged
anakin87 merged 6 commits intomainfrom
unify-hatch
Jun 6, 2025
Merged

chore: simplify Haystack Hatch scripts#9491
anakin87 merged 6 commits intomainfrom
unify-hatch

Conversation

@anakin87
Copy link
Copy Markdown
Member

@anakin87 anakin87 commented Jun 5, 2025

Related Issues

Proposed Changes:

  • introduce the Hatch scripts discussed in the original issue and adapt CI workflows
  • remove redundant docstring linting workflow
  • update ruff pre-commit and configure it to auto-fix if possible
  • update the contributing guidelines with the new scripts

How did you test it?

CI; local tests

Notes for the reviewer

I'll open similar PRs in experimental and core-integrations

Checklist

  • I have read the contributors guidelines and the code of conduct
  • I have updated the related issue with new insights and changes
  • I added unit tests and updated the docstrings
  • I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test: and added ! in case the PR includes breaking changes.
  • I documented my code
  • I ran pre-commit hooks and fixed any issue

@coveralls
Copy link
Copy Markdown
Collaborator

coveralls commented Jun 5, 2025

Pull Request Test Coverage Report for Build 15471784251

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 90.419%

Totals Coverage Status
Change from base Build 15468967841: 0.02%
Covered Lines: 11485
Relevant Lines: 12702

💛 - Coveralls

@anakin87 anakin87 added the ignore-for-release-notes PRs with this flag won't be included in the release notes. label Jun 5, 2025
@anakin87 anakin87 changed the title try unifying hatch scripts chore: simplify Haystack Hatch scripts Jun 5, 2025
@anakin87 anakin87 marked this pull request as ready for review June 5, 2025 16:01
@anakin87 anakin87 requested review from a team as code owners June 5, 2025 16:01
@anakin87 anakin87 requested review from dfokina, mpangrazzi and vblagoje and removed request for a team and vblagoje June 5, 2025 16:01
Copy link
Copy Markdown
Contributor

@mpangrazzi mpangrazzi left a comment

Choose a reason for hiding this comment

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

Looks good!

@anakin87 anakin87 merged commit 12665ad into main Jun 6, 2025
26 checks passed
@anakin87 anakin87 deleted the unify-hatch branch June 6, 2025 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ignore-for-release-notes PRs with this flag won't be included in the release notes. topic:build/distribution topic:CI topic:DX Developer Experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants