Skip to content

Conversation

@tassoevan
Copy link
Contributor

@tassoevan tassoevan commented Aug 7, 2022

Proposed changes (including videos or screenshots)

The current amount of ESLint warning messages is overwhelming to properly debug serious issues. This PR aims to reduce them to a sane amount.

Issue(s)

Steps to test or reproduce

Further comments

I couldn't go further as many issues regarding type declarations are too complex to solve without more information.

@tassoevan tassoevan force-pushed the fix/eslint-warnings branch from 07b9566 to 8770398 Compare August 7, 2022 07:00
@codecov
Copy link

codecov bot commented Aug 7, 2022

Codecov Report

Merging #26504 (8770398) into develop (f45b956) will decrease coverage by 0.00%.
The diff coverage is 62.85%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #26504      +/-   ##
===========================================
- Coverage    38.06%   38.06%   -0.01%     
===========================================
  Files          745      745              
  Lines        18628    18625       -3     
  Branches      1419     1412       -7     
===========================================
- Hits          7091     7089       -2     
+ Misses       11317    11313       -4     
- Partials       220      223       +3     
Flag Coverage Δ
e2e 38.06% <62.85%> (-0.01%) ⬇️

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

@tassoevan tassoevan marked this pull request as ready for review August 8, 2022 13:47
@tassoevan tassoevan requested review from a team as code owners August 8, 2022 13:47
@ggazzo ggazzo changed the title [FIX] ESLint warnings Chore: ESLint warnings Aug 8, 2022
@ggazzo ggazzo merged commit f74ba4e into develop Aug 8, 2022
@ggazzo ggazzo deleted the fix/eslint-warnings branch August 8, 2022 13:52
gabriellsh added a commit that referenced this pull request Aug 8, 2022
…homepage

* 'develop' of github.com:RocketChat/Rocket.Chat: (39 commits)
  Chore: ESLint warnings (#26504)
  Chore: Remove settings Fibers usage (#26465)
  Chore: Refactor create-target-channel util (#26493)
  Chore: useEndpointData deprecation (#26494)
  [NEW] Surface featured apps endpoint  (#26416)
  [FIX] Request at least one field in the payload of `/v1/users.setStatus` (#26490)
  Chore: Exit process on `unhandledRejection` on CI (#26467)
  [FIX] Clear push token on save user password (#26466)
  [FIX] Undefined MediaDevices error on HTTP (#26396)
  Chore: Codecov threshold (#26477)
  Chore: Tests intermitences (#26464)
  [FIX] Don't give errors on outbound voip call Request Terminated (#26373)
  Chore: Use Docker compose on CI (#26437)
  [FIX] DialPad call button from end to center (#26459)
  Chore: Parallelize e2e tests (#26390)
  [IMPROVE] use enter key to call using DialPad (#26454)
  Chore: Accounts/token to TS (#26434)
  Chore: Purge some unused modules (#26447)
  i18n: Language update from LingoHub 🤖 on 2022-08-01Z (#26429)
  Chore: Add end-to-end tests to teams listing in the `directory` endpoint (#26347)
  ...
@murtaza98 murtaza98 mentioned this pull request Sep 2, 2022
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.

2 participants