Skip to content

Versioning of sculptor and Pictor#57

Open
nicosammito wants to merge 9 commits intomainfrom
feat/versioning
Open

Versioning of sculptor and Pictor#57
nicosammito wants to merge 9 commits intomainfrom
feat/versioning

Conversation

@nicosammito
Copy link
Contributor

No description provided.

@nicosammito nicosammito self-assigned this Feb 16, 2026
@github-actions
Copy link

github-actions bot commented Feb 16, 2026

GitLab Pipeline Action

General information

Link to pipeline: https://gitlab.com/code0-tech/development/sculptor/-/pipelines/2327622726

Status: Passed
Duration: 2 minutes

Copy link
Member

@Taucher2003 Taucher2003 left a comment

Choose a reason for hiding this comment

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

Does this have to be done with environment variables? Feels a bit wrong, but if that is the best option for a next app, then its fine.

Copy link
Member

Choose a reason for hiding this comment

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

Which values here are required?

NEXT_PUBLIC_SCULPTOR_VERSION and NEXT_PUBLIC_PICTOR_VERSION are managed by the set-edition script. Are the other two variables required or what are they used for?

More importantly, we don't know the SAGITTARIUS_GRAPHQL_URL during build, so this variable must be overridable at runtime without rebuilding.

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.

2 participants

Comments