refactor: remove 'incubating' branding, update docs & packaging#318
refactor: remove 'incubating' branding, update docs & packaging#318imbajin merged 4 commits intoapache:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR removes obsolete “incubating” branding across the repo, updating repository links and release packaging paths to reflect the post-incubation Apache HugeGraph AI repository identity.
Changes:
- Replace
apache/incubator-hugegraph-ailinks withapache/hugegraph-aiacross docs, metadata, and GitHub templates/workflows. - Update Apache release script artifact naming and ASF dist SVN upload path (remove
-incubatingand/incubator/). - Remove
DISCLAIMERand updateNOTICEmetadata.
Reviewed changes
Copilot reviewed 19 out of 19 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| vermeer-python-client/pyproject.toml | Update repository URL to non-incubator repo. |
| vermeer-python-client/README.md | Update Vermeer link + clone/install instructions + Issues link. |
| scripts/apache-release.sh | Remove -incubating from source package name/prefix; update ASF SVN path out of /incubator/. |
| pyproject.toml | Update repository/bug tracker/changelog URLs to non-incubator repo. |
| hugegraph-python-client/pyproject.toml | Update homepage/repository/bug tracker URLs to non-incubator repo. |
| hugegraph-python-client/README.md | Update clone instructions and Issues link to non-incubator repo. |
| hugegraph-ml/pyproject.toml | Update repository URL to non-incubator repo. |
| hugegraph-ml/README.md | Update clone instructions to non-incubator repo. |
| hugegraph-llm/src/hugegraph_llm/demo/rag_demo/vector_graph_block.py | Update template link to point at non-incubator repo. |
| hugegraph-llm/pyproject.toml | Update repository/bug tracker URLs to non-incubator repo. |
| hugegraph-llm/README.md | Update DeepWiki, clone instructions, and template/quick-start links to non-incubator repo. |
| docker/env.template | Update example path to non-incubator repo directory name. |
| README.md | Update DeepWiki, clone instructions, related-project links, contributors graph, and issues link to non-incubator repos. |
| NOTICE | Remove “(incubating)” and update copyright year. |
| DISCLAIMER | Remove obsolete incubator DISCLAIMER file. |
| .github/workflows/sync.yml | Update fork-sync upstream repo name to non-incubator repo. |
| .github/ISSUE_TEMPLATE/question_ask.yml | Update issues search link to non-incubator repo. |
| .github/ISSUE_TEMPLATE/feature_request.yml | Update feature-search links to non-incubator repo. |
| .github/ISSUE_TEMPLATE/bug_report.yml | Update issues search link to non-incubator repo. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
And mark the reminder for another TODO: https://github.com/apache/hugegraph-ai/actions/runs/23279377962/workflow?pr=318#L22 |
| # The current CI workflow still boots HugeGraph 1.3.0, where | ||
| # `/metrics/system` returns 500 in GitHub Actions. Keep the check for | ||
| # newer servers and leave the workflow upgrade as a separate TODO. | ||
| server_version = tuple(self.client.client.cfg.version) |
There was a problem hiding this comment.
cfg.version stays empty when /versions cannot be parsed or fetched, and tuple([]) >= (1, 5, 0) is simply False, so we silently skip the /metrics/system assertion even on environments where we meant to keep coverage.
Could we make the incompatibility explicit instead of passing the test without checking that endpoint?
| server_version = tuple(self.client.client.cfg.version) | |
| server_version = tuple(self.client.client.cfg.version) | |
| self.assertTrue(server_version, "failed to detect HugeGraph server version") | |
| if server_version < (1, 5, 0): | |
| self.skipTest("HugeGraph < 1.5.0 returns 500 for /metrics/system in CI") | |
| system_metrics = self.metrics.get_system_metrics() | |
| self.assertIsInstance(system_metrics, dict) |
- extract the system metrics version gate into a small helper - fail fast when HugeGraph version detection is missing - skip legacy servers explicitly instead of silently skipping coverage - add focused regression tests for missing, legacy, and supported versions
- rename the NOTICE header to Apache HugeGraph AI - keep the NOTICE content aligned with this repository and release artifacts - replace the broken python-client feedback email link with a mailto target
imbajin
left a comment
There was a problem hiding this comment.
Thanks for ur contribution~ Link a TODO issue for the client
DISCLAIMERfileNOTICEproject name and copyright yearapache/incubator-hugegraph-ailinks withapache/hugegraph-ai