8000 Update RELEASE.md by Eddykasp · Pull Request #1132 · eclipse-elk/elk · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Update RELEASE.md #1132

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update RELEASE.md #1132

wants to merge 2 commits into from

Conversation

Eddykasp
Copy link
Contributor

Remove instruction to remove website stage from Jenkinsfile.

It turns out Jenkins always uses the Jenkinsfile from the master branch, so removing the call on the release branch doesn't even have any effect.

This PR serves as a discussion starting point for how we would like to handle this in the future.

Remove instruction to remove website stage from Jenkinsfile

Signed-off-by: Max Kasperowski <Eddykasp@users.noreply.github.com>
@Eddykasp Eddykasp requested a review from soerendomroes March 13, 2025 14:42
Signed-off-by: Max Kasperowski <Eddykasp@users.noreply.github.com>
@Eddykasp Eddykasp added this to the Release 0.10.1 milestone May 12, 2025
Copy link
Contributor
@soerendomroes soerendomroes left a comment

Choose a reason for hiding this comment

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

Why is the remove Website stuff no longer necessary?

@Eddykasp
Copy link
Contributor Author

Why is the remove Website stuff no longer necessary?

Because it doesn't do anything and probably never did. The Jenkins build uses the Jenkinsfile defined on the master branch for builds on any branch. So modifying the file only on the branch doesn't actually prevent the website from being built.

@soerendomroes
Copy link
Contributor

Maybe we jsut need a variable to build the website.

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
0