squash! PATCH: [TMP] AR-3427 Limit chat.syncMessages to syncing rec…
#78
Annotations
10 errors and 12 warnings
|
apps/meteor/app/file-upload/server/lib/requests.ts#L4
There should be at least one empty line between import groups
|
|
apps/meteor/app/file-upload/server/lib/requests.ts#L5
`@rocket.chat/logger` import should occur before import of `@rocket.chat/models`
|
|
|
|
apps/meteor/app/lib/server/functions/setEmail.ts#L20
'email' is assigned a value but never used
|
|
apps/meteor/app/oembed/server/server.ts#L99
Delete `·`
|
|
apps/meteor/app/oembed/server/server.ts#L104
Replace `(process.env.SPEAKEASY_HTTP_HEADER_NAME·&&·process.env.SPEAKEASY_HTTP_HEADER_VALUE)` with `process.env.SPEAKEASY_HTTP_HEADER_NAME·&&⏎↹↹process.env.SPEAKEASY_HTTP_HEADER_VALUE`
|
|
apps/meteor/server/models/raw/Messages.ts#L869
Variable name `duration_30_days_in_ms` must match one of the following formats: camelCase, UPPER_CASE, PascalCase
|
|
|
|
apps/meteor/server/models/raw/Messages.ts#L870
Variable name `date_30_days_ago` must match one of the following formats: camelCase, UPPER_CASE, PascalCase
|
|
|
|
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/cache@v3, actions/setup-node@v3.7.0. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
packages/fuselage-ui-kit/jest.setup.ts#L4
Do not use "@ts-ignore" because it alters compilation errors
|
|
apps/meteor/app/lib/server/lib/debug.js#L84
Consider using the native Array.prototype.forEach() or Object.entries().forEach()
|
|
apps/meteor/app/lib/server/oauth/oauth.js#L42
Consider using the native Array.prototype.includes()
|
|
apps/meteor/app/lib/server/startup/rateLimiter.js#L65
Consider using the native Array.prototype.forEach() or Object.entries().forEach()
|
|
apps/meteor/app/markdown/lib/hljs.js#L10
Async arrow function has a complexity of 173. Maximum allowed is 31
|
|
apps/meteor/app/meteor-accounts-saml/server/lib/SAML.ts#L81
Static async method 'insertOrUpdateSAMLUser' has a complexity of 32. Maximum allowed is 31
|
|
apps/meteor/app/reactions/client/init.ts#L32
Forbidden non-null assertion
|
|
apps/meteor/app/threads/client/messageAction/replyInThread.ts#L24
Forbidden non-null assertion
|
|
apps/meteor/app/utils/lib/restrictions.ts#L9
Consider using the native Array.prototype.map()
|
|
apps/meteor/app/utils/lib/restrictions.ts#L20
Consider using the native Array.prototype.map()
|
|
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/
|
The logs for this run have expired and are no longer available.
Loading