Skip to content

Conversation

@himani2411
Copy link
Contributor

@himani2411 himani2411 commented Dec 29, 2025

Description of changes

  • Add validation for input of pcluster-version and awsbatch-cli

Tests

References

  • Link to impacted open issues.
  • Link to related PRs in other packages (i.e. cookbook, node).
  • Link to documentation useful to understand the changes.

Checklist

  • Make sure you are pointing to the right branch.
  • If you're creating a patch for a branch other than develop add the branch name as prefix in the PR title (e.g. [release-3.6]).
  • Check all commits' messages are clear, describing what and why vs how.
  • Make sure to have added unit tests or integration tests to cover the new/modified code.
  • Check if documentation is impacted by this change.

Please review the guidelines for contributing and Pull Request Instructions.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@himani2411 himani2411 requested a review from a team as a code owner December 29, 2025 17:18
@himani2411 himani2411 added the skip-changelog-update Disables the check that enforces changelog updates in PRs label Dec 29, 2025
@himani2411 himani2411 requested a review from a team as a code owner December 29, 2025 17:18
@himani2411 himani2411 added the 3.x label Dec 29, 2025
@codecov
Copy link

codecov bot commented Dec 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.18%. Comparing base (820e0d9) to head (b079661).
⚠️ Report is 33 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #7178   +/-   ##
========================================
  Coverage    90.18%   90.18%           
========================================
  Files          183      183           
  Lines        16565    16565           
========================================
  Hits         14939    14939           
  Misses        1626     1626           
Flag Coverage Δ
unittests 90.18% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@himani2411 himani2411 enabled auto-merge (rebase) December 29, 2025 17:27
@himani2411 himani2411 merged commit d990389 into aws:develop Dec 29, 2025
48 of 51 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3.x skip-changelog-update Disables the check that enforces changelog updates in PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants