Skip to content

Conversation

@adrianlyjak
Copy link
Contributor

No description provided.

@adrianlyjak adrianlyjak force-pushed the adrian/changesets-attempt branch from c9e6475 to 70eeb20 Compare November 17, 2025 17:44
Copy link
Member

@AstraBert AstraBert left a comment

Choose a reason for hiding this comment

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

lgtm, this is way cleaner :))

"name": "llama-index-utils-workflow",
"version": "0.5.0",
"private": "true",
"private": "false",
Copy link
Member

Choose a reason for hiding this comment

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

I don't know if public or private makes a big difference here, since we don't publish this as a JS package nevertheless, but the ratio behind setting it as private is because this is not a JS package and should not be interpreted as such by changesets

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The release process is completely run by our script, so this doesn't get published.

Maybe silly, but I'm reading the package.json private value in changesets.py to infer whether the package should be published (Could instead just treat all folders with both pyproject.toml and package.json as ones that should be published)

Copy link
Member

Choose a reason for hiding this comment

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

Oh yeah, makes sense then, fine to keep it as public :))

@adrianlyjak adrianlyjak marked this pull request as ready for review November 17, 2025 18:07
@adrianlyjak adrianlyjak merged commit 939e41d into clelia/changesets-attempt Nov 17, 2025
17 checks passed
@adrianlyjak adrianlyjak deleted the adrian/changesets-attempt branch November 17, 2025 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants