Skip to content

Comments

Refactor docs and pypi publish github actions#1178

Merged
vkehfdl1 merged 7 commits intomainfrom
HotFix/#1177
Nov 13, 2025
Merged

Refactor docs and pypi publish github actions#1178
vkehfdl1 merged 7 commits intomainfrom
HotFix/#1177

Conversation

@vkehfdl1
Copy link
Contributor

close #1177

@vkehfdl1 vkehfdl1 requested review from bwook00 and Copilot November 10, 2025 04:44
@vkehfdl1 vkehfdl1 enabled auto-merge (squash) November 10, 2025 04:44
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modernizes the release workflow by removing the VERSION file-based versioning in favor of dynamically setting versions from GitHub release tags, updates the publish workflow to use UV package manager instead of pip/twine, and fixes the documentation requirements path. However, there are critical configuration issues that will prevent successful builds.

  • Removes the VERSION file and transitions to tag-based versioning
  • Updates the GitHub Actions publish workflow to use UV and trigger on releases instead of main branch pushes
  • Changes docs requirements.txt path from ./autorag to .

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.

File Description
docs/requirements.txt Updates editable install path (but incorrectly points to current directory instead of parent)
autorag/VERSION Removes version file (but pyproject.toml still references it causing build failure)
.gitignore Adds .vscode/ directory to ignored files
.github/workflows/publish.yml Replaces push-triggered workflow with release-triggered workflow using UV (but has version injection issues)
Comments suppressed due to low confidence (1)

autorag/VERSION:1

  • The VERSION file is being deleted, but pyproject.toml still references it at line 105 in [tool.setuptools.dynamic] section: version = { file = ["autorag/VERSION"] }. This will cause build failures since setuptools won't be able to find the VERSION file to dynamically determine the package version. Either keep this file or update pyproject.toml to remove the dynamic version configuration.

bwook00
bwook00 previously approved these changes Nov 10, 2025
Copy link
Contributor

@bwook00 bwook00 left a comment

Choose a reason for hiding this comment

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

LGTM

@bwook00
Copy link
Contributor

bwook00 commented Nov 11, 2025

@vkehfdl1 Why did that test fail again? bug..?

@vkehfdl1 vkehfdl1 disabled auto-merge November 13, 2025 05:14
@vkehfdl1 vkehfdl1 merged commit 9d7da83 into main Nov 13, 2025
3 of 5 checks passed
@vkehfdl1 vkehfdl1 deleted the HotFix/#1177 branch November 13, 2025 05:14
vkehfdl1 added a commit that referenced this pull request Nov 14, 2025
* Refactor docs and pypi publish github actions (#1178)

* ignore .vscode folder in .gitignore

* Fix install of AutoRAG from the local path in docs/requirements.txt

* Fix pypi package dir path

* refactor publish to the pypi

* delete version information in __init__.py and optimize imports

* Fix publish.yml

* fix conf.py and test.yml

* remove dynamic version and replace build backend to hatchling. add ty… (#1186)

* remove dynamic version and replace build backend to hatchling. add ty and deptry

* recognize tests folder

* change pytest ini options

* disable delete tests

* fix version dependency in cli.py and updates the resources chroma in tests
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.

[HotFix] Fix documentation build and pypi build in github actions

2 participants