fix: imported fixes #25279

Triggered via pull request September 10, 2024 19:55

julio-cfa

synchronize #33246

imported-fixes-2024-09-09

Status Success

Total duration 57m 16s

Artifacts 12

ci.yml

on: pull_request

⚙️ Variables Setup

🚀 Notify external services - draft

📦 Build Packages

19m 13s

Matrix: 🔎 Code Check / code-check

📦 Meteor Build - coverage

6m 4s

🔨 Test Unit / Unit Tests

3m 44s

Matrix: 🚢 Build Docker Images for Testing

🚢 Build Docker Image (preview)

Matrix: 🔨 Test API (EE) / test

Matrix: 🔨 Test API (CE) / test

Matrix: 🔨 Test UI (EE) / test

Matrix: 🔨 Test UI (CE) / test

✅ Tests Done

📦 Meteor Build - official

Matrix: 🚢 Build Docker Images for Production

🚀 Publish build assets

Matrix: 🚀 Publish Docker Image (main)

Matrix: 🚀 Publish Docker Image (services)

🚀 Notify external services

trigger-dependent-workflows

Update Version Durability / update-versions

Annotations

56 warnings

📦 Build Packages

The following actions use a deprecated Node.js version and will be forced to run on node20: actions/cache@v3, actions/[email protected]. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

🔨 Test Unit / Unit Tests

The following actions use a deprecated Node.js version and will be forced to run on node20: actions/cache@v3, actions/[email protected], codecov/codecov-action@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

📦 Meteor Build - coverage

🚢 Build Docker Images for Testing (official)

The following actions use a deprecated Node.js version and will be forced to run on node20: docker/login-action@v2, actions/cache@v3, actions/[email protected]. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

🔎 Code Check / Code Lint

🔎 Code Check / Code Lint: apps/meteor/app/channel-settings/server/functions/saveRoomTopic.ts#L25

Forbidden non-null assertion

🔎 Code Check / Code Lint: apps/meteor/app/lib/server/lib/debug.js#L84

Consider using the native Array.prototype.forEach() or Object.entries().forEach()

🔎 Code Check / Code Lint: apps/meteor/app/lib/server/lib/notifyListener.ts#L379

Forbidden non-null assertion

🔎 Code Check / Code Lint: apps/meteor/app/lib/server/lib/notifyListener.ts#L383

Forbidden non-null assertion

🔎 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/livechat/server/lib/sendTranscript.ts#L25

Async function 'sendTranscript' has a complexity of 35. Maximum allowed is 31

🔎 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 33. Maximum allowed is 31

🔎 Code Check / Code Lint: apps/meteor/app/reactions/client/init.ts#L32

Forbidden non-null assertion

🚢 Build Docker Images for Testing (alpine)

🔎 Code Check / TypeScript

🔨 Test UI (CE) / MongoDB 4.4 (3/4)