Skip to content

Conversation

@skrawcz
Copy link
Contributor

@skrawcz skrawcz commented Dec 28, 2025

We need to add it to the README that is linked in the package.

This also adds this to all the other packages in this repo.

Changes

  • adds incubator disclaimer to README files.

How I tested this

  • locally

Notes

Checklist

  • PR has an informative and human-readable title (this will be pulled into the release notes)
  • Changes are limited to a single goal (no scope creep)
  • Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future TODOs are captured in comments
  • Project documentation has been updated if adding/changing functionality.

We need to add it to the README that is linked in the package.

This also adds this to all the other packages in this repo.
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 adds the Apache Software Foundation incubator disclaimer to README files across the repository, addressing issue #1421. The disclaimer informs users that Apache Hamilton is currently undergoing incubation at the ASF and has not yet been fully endorsed.

  • Adds identical incubator disclaimer to four README files
  • Disclaimer uses proper markdown blockquote formatting
  • Placement is consistent across all affected files (after title, before main content)

Reviewed changes

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

File Description
ui/sdk/README.md Adds incubator disclaimer to the Hamilton UI SDK README
dev_tools/language_server/README.md Adds incubator disclaimer to the Hamilton Language Server README
contrib/README.md Adds incubator disclaimer to the contributions package README
README.md Adds incubator disclaimer to the main repository README

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@jernejfrank jernejfrank left a comment

Choose a reason for hiding this comment

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

lgtm

@skrawcz skrawcz merged commit 9fd1bda into main Dec 29, 2025
8 of 20 checks passed
@skrawcz skrawcz deleted the stefan/fix-1421 branch December 29, 2025 01:00
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.

3 participants