These files are missing Apache License headers. ./CONTRIBUTING.md ./MANIFEST.in ./PKG-INFO ./README.md ./pyproject.toml ./rat.txt ./setup.cfg ./hamilton/experimental/databackend.py (this is ok because it is mentioned in the LICENSE file) ./sf_hamilton.egg-info/PKG-INFO ./sf_hamilton.egg-info/SOURCES.txt ./sf_hamilton.egg-info/dependency_links.txt ./sf_hamilton.egg-info/entry_points.txt ./sf_hamilton.egg-info/requires.txt ./sf_hamilton.egg-info/top_level.txt
On Sun, 14 Sept 2025 at 18:35, PJ Fanning <[email protected]> wrote: > > The LICENSE in the sf-hamilton file appears wrong. It mentions > contrib/hamilton/contrib/user/skrawcz/customize_embeddings/__init__.py > but I can't find that file in the tar.gz. > > I would also recommend that the directory inside the tar.gz omits the > rc0 bit. You can't modify the file after the release vote so you won't > be able to remove the rc0 bit when you do release. > > The next vote should be for rc1. And if that fails, rc2, etc. > I would recommend that the next release starts with rc1. This is more > usual for ASF release votes. > > On Sun, 14 Sept 2025 at 18:28, PJ Fanning <[email protected]> wrote: > > > > The rule about incubating in the name is at: > > https://incubator.apache.org/guides/releasemanagement.html > > > > <snip> > > Here is a minimal set of requirements, when using the work in progress > > disclaimer, a podlings release must abide by: > > > > Include the word incubating in the release file name. > > > > Include an ASF LICENSE and NOTICE file. > > > > Have valid checksums or signatures. > > > > Be placed in the correct place on the ASF’s infrastructure. > > > > Have a KEYS file to validate the release. > > > > Other issues, such as: > > > > Missing ASF headers. > > > > Missing license information. > > > > Included unexpected binary code. > > > > Including code of unknown origin. > > </snip> > > > > On Sun, 14 Sept 2025 at 18:25, PJ Fanning <[email protected]> wrote: > > > > > > I'm not sure what process is being followed. Many podlings have a > > > document describing the release process that they follow. > > > I want to highlight one important item for podlings. The release must > > > be approved by a vote in the podling mailing list. Requires at least > > > 72 hours of voting and a min of 3 +1s from PPMC members. But you also > > > need a 2nd vote after this on the Incubator general mailing list where > > > the IPMC members vote separately. Requires at least 72 hours of voting > > > and a min of 3 +1s from IPMC members. > > > > > > On Sun, 14 Sept 2025 at 18:19, PJ Fanning <[email protected]> wrote: > > > > > > > > The source release which is required must have 'incubating' in the name. > > > > If you want to also release an sf-hamilton - then you can have 2 tar gz > > > > files. > > > > > > > > Here is an actual vote thread to look at. > > > > https://lists.apache.org/thread/b14519zt9v2s9gvpg6jo0wzhblo7drhx > > > > > > > > It has a description of the voting rules and the checks that are > > > > required (or suggested) for voters to perform. > > > > > > > > On Sun, 14 Sept 2025 at 18:16, PJ Fanning <[email protected]> wrote: > > > > > > > > > > Votes for releases are usually proceeded by discussion threads. > > > > > I would suggest that you look at other podling mailing lists to see > > > > > how podlings are administered. > > > > > List of podlings > > > > > https://incubator.apache.org/clutch/ > > > > > Mailing list reading > > > > > https://lists.apache.org/ > > > > > > > > > > On 2025/09/14 16:38:18 Stefan Krawczyk wrote: > > > > > > Okay I think I find one issue. So -1. > > > > > > > > > > > > The code literally has RC in version.py. if we just literally move > > > > > > this > > > > > > source to releases then this won't fly. I'll need to change this. > > > > > > Please > > > > > > continue to verify. I am assuming there might be one or two more > > > > > > things to > > > > > > correct. Will create another RC candidate and vote after this one > > > > > > concludes. > > > > > > > > > > > > > > > > > > > > > > > > On Sat, Sep 13, 2025, 10:26 PM Stefan Krawczyk > > > > > > <[email protected]> > > > > > > wrote: > > > > > > > > > > > > > Hi all, > > > > > > > > > > > > > > This is a call for a vote on releasing Apache hamilton > > > > > > > 1.89.0-incubating, > > > > > > > release candidate 0. > > > > > > > > > > > > > > This release includes the following changes: > > > > > > > - Too many to list. Will build tooling to help automate this. > > > > > > > > > > > > > > The artifacts for this release candidate can be found at: > > > > > > > https://dist.apache.org/repos/dist/dev/incubator/hamilton/1.89.0-RC0 > > > > > > > > > > > > > > The Git tag to be voted upon is: > > > > > > > v1.89.0 (based off of this PR > > > > > > > <https://github.com/apache/hamilton/pull/1378>) > > > > > > > > > > > > > > The release hash (can use git tag) will be provided once we merge > > > > > > > a commit > > > > > > > with version 1.89.0 to main. > > > > > > > > > > > > > > Release artifacts are signed with the following key: > > > > > > > > > > > > > > 25E1C6FA3B71D486DC46BD3630C8F2B2CC329C0B > > > > > > > > > > > > > > The KEYS file is available at: > > > > > > > https://downloads.apache.org/incubator/hamilton/KEYS > > > > > > > > > > > > > > Please download, verify, and test the release candidate. > > > > > > > > > > > > > > The vote will run for a minimum of 72 hours. > > > > > > > Please vote: > > > > > > > > > > > > > > [ ] +1 Release this package as Apache hamilton 1.89.0-incubating > > > > > > > [ ] +0 No opinion > > > > > > > [ ] -1 Do not release this package because... (Please provide a > > > > > > > reason) > > > > > > > > > > > > > > On behalf of the Apache Hamilton PPMC, > > > > > > > > > > > > > > Stefan Krawczyk > > > > > > > > > > > > > > P.S. @Mentors with our project, we literally commit the version > > > > > > > into the > > > > > > > source code. This means that we will build releases off of > > > > > > > branches and > > > > > > > then if all good merge into main. Is that kosher? or not? > > > > > > > > > > > > >
