Skip to content

Conversation

Toflar
Copy link
Member

@Toflar Toflar commented Sep 29, 2025

Noticed some hardcoded 4.x references and thought that it would be good making those dynamic to config. Also updated the config to represent the current state of things.

Copy link
Contributor

@fritzmg fritzmg left a comment

Choose a reason for hiding this comment

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

Ah, I thought I already changed this at those spots. Though imho we don't really need to overcomplicate it with so many different versions - installing should just reference the newest stable version imho.

I'll adjust the PR accordingly.

currentReleaseCandidate: '5.6'
currentContaoLtsVersion: '5.3'
currentContaoVersion: '5.6'
currentReleaseCandidate: '5.7'
Copy link
Contributor

Choose a reason for hiding this comment

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

This won't work - it will then display 5.7@RC in the docs, which you will not be able to install.

Copy link
Member Author

Choose a reason for hiding this comment

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

Kk, feel free to update that too then

@fritzmg fritzmg merged commit 2554079 into main Sep 29, 2025
@fritzmg fritzmg deleted the feature/lts-version branch September 29, 2025 14:39
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.

2 participants