Skip to content

fix: Skip conditionalLockAgent locking path for bot agents#40771

Merged
dionisio-bot[bot] merged 1 commit into
release-8.4.3from
backport-8.4.3-40635
Jun 2, 2026
Merged

fix: Skip conditionalLockAgent locking path for bot agents#40771
dionisio-bot[bot] merged 1 commit into
release-8.4.3from
backport-8.4.3-40635

Conversation

@dionisio-bot
Copy link
Copy Markdown
Contributor

@dionisio-bot dionisio-bot Bot commented Jun 2, 2026

Backport of #40635

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: KevLehman <11577696+KevLehman@users.noreply.github.com>
Co-authored-by: Kevin Aleman <kaleman960@gmail.com>
@dionisio-bot dionisio-bot Bot requested review from a team as code owners June 2, 2026 16:50
@dionisio-bot dionisio-bot Bot added the backport Used to inform backported PR label Jun 2, 2026
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jun 2, 2026

🦋 Changeset detected

Latest commit: bbd056e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@rocket.chat/meteor Patch
@rocket.chat/core-typings Patch
@rocket.chat/rest-typings Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.95%. Comparing base (9c1c81c) to head (bbd056e).
⚠️ Report is 2 commits behind head on release-8.4.3.

Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                @@
##           release-8.4.3   #40771      +/-   ##
=================================================
- Coverage          69.96%   69.95%   -0.02%     
=================================================
  Files               3307     3307              
  Lines             120587   120588       +1     
  Branches           21601    21629      +28     
=================================================
- Hits               84373    84361      -12     
- Misses             32916    32929      +13     
  Partials            3298     3298              
Flag Coverage Δ
e2e 59.64% <ø> (+0.01%) ⬆️
e2e-api 46.36% <ø> (-0.92%) ⬇️
unit 70.73% <100.00%> (+<0.01%) ⬆️

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@KevLehman KevLehman added the stat: QA assured Means it has been tested and approved by a company insider label Jun 2, 2026
@dionisio-bot
Copy link
Copy Markdown
Contributor Author

dionisio-bot Bot commented Jun 2, 2026

Looks like this PR is not ready to merge, because of the following issues:

  • This PR is missing the required milestone or project

Please fix the issues and try again

If you have any trouble, please check the PR guidelines

@dionisio-bot dionisio-bot Bot merged commit 7db101f into release-8.4.3 Jun 2, 2026
47 of 48 checks passed
@dionisio-bot dionisio-bot Bot deleted the backport-8.4.3-40635 branch June 2, 2026 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Used to inform backported PR community stat: QA assured Means it has been tested and approved by a company insider

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants