Skip to content

fix: cleaning the room filter between navigation on the admin section #20280

fix: cleaning the room filter between navigation on the admin section

fix: cleaning the room filter between navigation on the admin section #20280

Triggered via pull request May 7, 2024 19:38
Status Failure
Total duration 17m 10s
Artifacts 1

ci.yml

on: pull_request
⚙️ Variables Setup
6s
⚙️ Variables Setup
🚀 Notify external services - draft
3s
🚀 Notify external services - draft
📦 Build Packages
1m 28s
📦 Build Packages
Matrix: 🔎 Code Check / code-check
📦 Meteor Build - coverage
7m 31s
📦 Meteor Build - coverage
🔨 Test Unit  /  Unit Tests
1m 29s
🔨 Test Unit / Unit Tests
Matrix: 🚢 Build Docker Images for Testing
🚢 Build Docker Image (preview)
0s
🚢 Build Docker Image (preview)
Matrix: 🔨 Test API (EE) / test
Waiting for pending jobs
Matrix: 🔨 Test API (CE) / test
Waiting for pending jobs
Matrix: 🔨 Test UI (EE) / test
Waiting for pending jobs
Matrix: 🔨 Test UI (CE) / test
Waiting for pending jobs
✅ Tests Done
0s
✅ Tests Done
📦 Meteor Build - official
0s
📦 Meteor Build - official
Matrix: 🚢 Build Docker Images for Production
🚀 Publish build assets
0s
🚀 Publish build assets
Matrix: 🚀 Publish Docker Image (main)
Matrix: 🚀 Publish Docker Image (services)
🚀 Notify external services
0s
🚀 Notify external services
trigger-dependent-workflows
0s
trigger-dependent-workflows
Fit to window
Zoom out
Zoom in

Annotations

9 errors and 20 warnings
🔎 Code Check / Code Lint
@rocket.chat/meteor#lint: command (/home/runner/work/Rocket.Chat/Rocket.Chat/apps/meteor) /tmp/xfs-b4f7859f/yarn run lint exited (1)
🔎 Code Check / Code Lint: apps/meteor/tests/unit/client/views/admin/rooms/RoomsTable.spec.tsx#L11
There should be at least one empty line between import groups
🔎 Code Check / Code Lint: apps/meteor/tests/unit/client/views/admin/rooms/RoomsTable.spec.tsx#L12
`@rocket.chat/mock-providers` import should occur before import of `@rocket.chat/ui-contexts`
🔎 Code Check / Code Lint: apps/meteor/tests/unit/client/views/admin/rooms/RoomsTable.spec.tsx#L58
A function with a name starting with an uppercase letter should only be used as a constructor
🔎 Code Check / Code Lint: apps/meteor/tests/unit/client/views/admin/rooms/RoomsTable.spec.tsx#L90
A function with a name starting with an uppercase letter should only be used as a constructor
🔎 Code Check / Code Lint: apps/meteor/tests/unit/client/views/admin/rooms/RoomsTable.spec.tsx#L120
A function with a name starting with an uppercase letter should only be used as a constructor
🔎 Code Check / Code Lint: apps/meteor/tests/unit/client/views/admin/rooms/RoomsTable.spec.tsx#L136
A function with a name starting with an uppercase letter should only be used as a constructor
🔎 Code Check / Code Lint
Process completed with exit code 1.
⚙️ Variables Setup
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: Bhacaz/checkout-files@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
🚀 Notify external services - draft
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: Bhacaz/checkout-files@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
📦 Build Packages
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/cache@v3, actions/setup-node@v3.7.0. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
🔨 Test Unit / Unit Tests
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/cache@v3, actions/setup-node@v3.7.0, codecov/codecov-action@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
🔎 Code Check / TypeScript
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/cache@v3, actions/setup-node@v3.7.0. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
📦 Meteor Build - coverage
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/cache@v3, actions/setup-node@v3.7.0, actions/upload-artifact@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
🔎 Code Check / Code Lint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/cache@v3, actions/setup-node@v3.7.0. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
🔎 Code Check / Code Lint: apps/meteor/app/e2e/client/rocketchat.e2e.ts#L443
Prefer using an optional chain expression instead, as it's more concise and easier to read
🔎 Code Check / Code Lint: apps/meteor/app/lib/server/lib/debug.js#L83
Consider using the native Array.prototype.forEach() or Object.entries().forEach()
🔎 Code Check / Code Lint: apps/meteor/app/lib/server/oauth/oauth.js#L42
Consider using the native Array.prototype.includes()
🔎 Code Check / Code Lint: apps/meteor/app/lib/server/startup/rateLimiter.js#L65
Consider using the native Array.prototype.forEach() or Object.entries().forEach()
🔎 Code Check / Code Lint: apps/meteor/app/markdown/lib/hljs.js#L10
Async arrow function has a complexity of 173. Maximum allowed is 31
🔎 Code Check / Code Lint: apps/meteor/app/meteor-accounts-saml/server/lib/SAML.ts#L81
Static async method 'insertOrUpdateSAMLUser' has a complexity of 32. Maximum allowed is 31
🔎 Code Check / Code Lint: apps/meteor/app/utils/lib/restrictions.ts#L9
Consider using the native Array.prototype.map()
🔎 Code Check / Code Lint: apps/meteor/app/utils/lib/restrictions.ts#L20
Consider using the native Array.prototype.map()
🚢 Build Docker Images for Testing (official)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: docker/login-action@v2, actions/download-artifact@v3, actions/cache@v3, actions/setup-node@v3.7.0. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
🚢 Build Docker Images for Testing (alpine)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: docker/login-action@v2, actions/download-artifact@v3, actions/cache@v3, actions/setup-node@v3.7.0. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Deprecation notice: v1, v2, and v3 of the artifact actions
The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "build". Please update your workflow to use v4 of the artifact actions. Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/

Artifacts

Produced during runtime
Name Size
build
306 MB