Skip to content

Comments

Versioning of sculptor and Pictor#57

Merged
nicosammito merged 9 commits intomainfrom
feat/versioning
Feb 20, 2026
Merged

Versioning of sculptor and Pictor#57
nicosammito merged 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/2340529865

Status: Passed
Duration: 4 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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We decided to use a Nginx server for that

@nicosammito nicosammito merged commit 57e16d9 into main Feb 20, 2026
1 check passed
@nicosammito nicosammito deleted the feat/versioning branch February 20, 2026 22:40
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