Skip to content
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

Handling of issues for the CLR queue during the code freeze period. #329

Merged
merged 2 commits into from
Mar 4, 2025

Conversation

junpataleta
Copy link
Contributor

  • Update the Manage queues on continuous job so that improvements and new features in the CLR queue will not be held during the freeze period until the last week before the release (behaviour A1).

Note: Keeping the behaviour for

  • A3b (last week before release, all issues unrelated to the release will be held)
  • B1a (during the on-sync period, only bugs will be integrated)
  • During the freeze, improvements and new features will be excluded from being triaged for the CLR queue so that they will remain in the candidate queue and be held accordingly.

* During the freeze period up to the last week before the release,
improvements and new features in the CLR queue will not be held
(behaviour A1).

Note: Keeping the behaviour for
- A3b (last week before release, all issues unrelated to the
  release will be held)
- B1a (during on-sync period, only bugs will be integrated)
@junpataleta junpataleta force-pushed the clr_handling_during_freeze branch from 88c6c35 to 74be6c8 Compare February 27, 2025 22:59
@codecov-commenter
Copy link

codecov-commenter commented Feb 27, 2025

Codecov Report

Attention: Patch coverage is 0% with 8 lines in your changes missing coverage. Please review.

Project coverage is 22.65%. Comparing base (f92c381) to head (b32080e).

Files with missing lines Patch % Lines
...gration_mover/component_leads_integration_mover.sh 0.00% 8 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (f92c381) and HEAD (b32080e). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (f92c381) HEAD (b32080e)
remote_branch_checker 1 0
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #329      +/-   ##
==========================================
- Coverage   29.76%   22.65%   -7.11%     
==========================================
  Files          71       71              
  Lines        3215     3222       +7     
==========================================
- Hits          957      730     -227     
- Misses       2258     2492     +234     
Flag Coverage Δ
check_upgrade_savepoints 0.46% <0.00%> (-0.01%) ⬇️
checkstyle_manipulations 0.00% <0.00%> (ø)
compare_databases 2.38% <0.00%> (-0.01%) ⬇️
continuous_manage_queues_validation 0.71% <0.00%> (-0.01%) ⬇️
define_excluded 1.11% <0.00%> (-0.01%) ⬇️
detect_conflicts 1.48% <0.00%> (-0.01%) ⬇️
diff_extract_changes 0.00% <0.00%> (ø)
grunt_process 3.01% <0.00%> (-0.01%) ⬇️
illegal_whitespace 2.63% <0.00%> (-0.01%) ⬇️
list_valid_components 0.00% <0.00%> (ø)
mustache_lint 2.82% <0.00%> (-0.01%) ⬇️
mustache_lint_plugins 1.30% <0.00%> (-0.01%) ⬇️
php_lint 1.21% <0.00%> (-0.01%) ⬇️
prepare_npm_stuff 1.52% <0.00%> (-0.01%) ⬇️
remote_branch_checker ?
remote_branch_reporter 0.00% <0.00%> (ø)
setup 0.00% <0.00%> (ø)
thirdparty_check 1.45% <0.00%> (-0.01%) ⬇️
travis-branch-checker 0.00% <0.00%> (ø)
upgrade_external_backup 1.61% <0.00%> (-0.01%) ⬇️
verifications 0.00% <0.00%> (ø)
verify_commit_messages 2.63% <0.00%> (-0.01%) ⬇️
versions_check_set 5.02% <0.00%> (-0.02%) ⬇️

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.

@junpataleta junpataleta force-pushed the clr_handling_during_freeze branch from 74be6c8 to b32080e Compare February 28, 2025 00:00
@HuongNV13 HuongNV13 self-requested a review March 4, 2025 02:05
@HuongNV13
Copy link
Contributor

Thanks, Jun.
It looks good to me.
It seems like the failing checks are not related to this patch.
I'm happy to merge this one now!

@HuongNV13 HuongNV13 merged commit 3a1af67 into moodlehq:main Mar 4, 2025
47 of 51 checks passed
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.

3 participants