Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[release/v7.4] Fix seed max value for Container Linux CI#24543

Merged
TravisEz13 merged 2 commits into
PowerShell:release/v7.4PowerShell/PowerShell:release/v7.4from
TravisEz13:backport-24510TravisEz13/PowerShell:backport-24510Copy head branch name to clipboard
Nov 4, 2024
Merged

[release/v7.4] Fix seed max value for Container Linux CI#24543
TravisEz13 merged 2 commits into
PowerShell:release/v7.4PowerShell/PowerShell:release/v7.4from
TravisEz13:backport-24510TravisEz13/PowerShell:backport-24510Copy head branch name to clipboard

Conversation

@TravisEz13
Copy link
Copy Markdown
Member

Backport #24510

This pull request includes a change to the Invoke-InitializeContainerStage function in the tools/ci.psm1 file to improve the consistency of the seed value for PR builds and ensure it fits within 32 bits.

Changes to seed value handling:

  • Added logic to ensure the seed value does not exceed 32 bits by using a bitwise AND operation with Int32::MaxValue.

@TravisEz13 TravisEz13 added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Nov 4, 2024
@TravisEz13 TravisEz13 self-assigned this Nov 4, 2024
@TravisEz13 TravisEz13 merged commit 9a78378 into PowerShell:release/v7.4 Nov 4, 2024
@TravisEz13 TravisEz13 deleted the backport-24510 branch November 4, 2024 22:23
@microsoft-github-policy-service
Copy link
Copy Markdown
Contributor

microsoft-github-policy-service Bot commented Nov 4, 2024

📣 Hey @TravisEz13, how did we do? We would love to hear your feedback with the link below! 🗣️

🔗 https://aka.ms/PSRepoFeedback

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.