Skip to content

[bugfix] The version must be set in a separate stage, so it can be reused in the other calls #399

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

Merged
merged 1 commit into from
May 26, 2025

Conversation

FaramosCZ
Copy link
Contributor

@FaramosCZ FaramosCZ commented May 22, 2025

otherwise it's not available when the rest of the ENV block is processed and the ENV values relying on this variable are malformed.

This commit was mistakenly reverted as a part of revert of a different change

…-used in the other calls

otherwise it's not available when the rest of the ENV block is processed
and the ENV values relying on this variable are malformed.

This commit was mistakenly reverted as a part of revert of a different change
@FaramosCZ
Copy link
Contributor Author

[test]

Copy link

github-actions bot commented May 22, 2025

Testing Farm results

namecomposearchstatusstarted (UTC)timelogs
Fedora - 8.0Fedora-latestx86_64✅ passed23.05.2025 07:18:4312min 58stest pipeline
CentOS Stream 9 - 8.0CentOS-Stream-9x86_64✅ passed23.05.2025 07:18:389min 56stest pipeline
Fedora - 8.4Fedora-latestx86_64✅ passed23.05.2025 07:18:4412min 55stest pipeline
CentOS Stream 10 - 8.4CentOS-Stream-10x86_64✅ passed23.05.2025 07:18:368min 57stest pipeline
RHEL9 - 8.0RHEL-9.4.0-Nightlyx86_64✅ passed23.05.2025 07:18:3718min 49stest pipeline
RHEL9 - 8.4RHEL-9.4.0-Nightlyx86_64✅ passed23.05.2025 07:18:3720min 4stest pipeline
RHEL8 - 8.0RHEL-8.10.0-Nightlyx86_64✅ passed23.05.2025 07:18:3618min 30stest pipeline
CentOS Stream 9 - 8.4CentOS-Stream-9x86_64✅ passed23.05.2025 07:18:389min 11stest pipeline
RHEL10 - 8.4RHEL-10-Nightlyx86_64✅ passed23.05.2025 07:18:3516min 18stest pipeline

Copy link

github-actions bot commented May 22, 2025

Pull Request validation

Success

🟢 CI - All checks have passed
🟢 Review - Reviewed by a member
🟢 Approval - Changes were approved


Auto Merge

Failed

🔴 Pull Request has unsupported target branch master, expected branches are: ''

Success

🟢 Pull Request is not marked as draft and it's not blocked by dont-merge label
🟢 Pull Request meet requirements, title has correct form
🟢 Pull Request meet requirements, mergeable is true
🟢 Pull Request meet requirements, mergeable_state is clean

@FaramosCZ
Copy link
Contributor Author

[test]

1 similar comment
@FaramosCZ
Copy link
Contributor Author

[test]

Copy link
Member

@phracek phracek left a comment

Choose a reason for hiding this comment

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

LGTM.

@phracek phracek merged commit 10fd875 into sclorg:master May 26, 2025
10 checks passed
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