Skip to content

chore(lint): Rule adjustments and fix warnings #19612

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
logaretm merged 22 commits into from
Mar 10, 2026
Merged

chore(lint): Rule adjustments and fix warnings #19612

Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
f5135e7
chore: initial config and rule setup
logaretm Feb 2, 2026
a722d54
chore: cleanup eslint
logaretm Feb 2, 2026
b1d889a
chore: renable no floating promises
logaretm Feb 19, 2026
cef9a25
chore: re-enable prefer-optional-chain
logaretm Feb 19, 2026
03031d0
chore: enable no-unsafe-member-access
logaretm Feb 19, 2026
41b258d
chore: remove unused imports
logaretm Mar 9, 2026
ab2b8d2
ref: deleted unused debugger.info method
logaretm Mar 9, 2026
355ad20
lint: remove redundant undefineds
logaretm Mar 9, 2026
7cf7e2c
chore: disable several rules
logaretm Mar 9, 2026
d840f01
lint: fix unused vars by prefixing _
logaretm Mar 9, 2026
74d86ff
lint: remove jest rules since we use vitest
logaretm Mar 9, 2026
b8f68dd
lint: fix duplicate types
logaretm Mar 9, 2026
903df65
fix: format
logaretm Mar 9, 2026
fe828c5
chore: revert nx change
logaretm Mar 9, 2026
1e954ab
lint: configure await-thenable as warn, disable in tests/dev-packages
logaretm Mar 9, 2026
e953478
lint: fix await-thenable warnings
logaretm Mar 9, 2026
d3c7bad
lint: simplify optional chaining in client and transport files
logaretm Mar 10, 2026
96a4f31
chore: remove unused jest rules configs
logaretm Mar 10, 2026
52179c4
chore: no longer needed ignore
logaretm Mar 10, 2026
3fcaf65
lint: remove unused ignores
logaretm Mar 10, 2026
dbf0ee5
lint: revert exact prop types changes
logaretm Mar 10, 2026
e3b9475
lint: remove unused error catch param
logaretm Mar 10, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
ref: deleted unused debugger.info method
  • Loading branch information
@logaretm
logaretm committed Mar 10, 2026
commit ab2b8d225d3d9da5ecb3c778af516db672d1cd55
4 changes: 0 additions & 4 deletions packages/core/src/utils/debug-logger.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,10 +85,6 @@ function log(...args: Parameters<typeof console.log>): void {
_maybeLog('log', ...args);
}

function info(...args: Parameters<typeof console.info>): void {
_maybeLog('info', ...args);
}
Comment on lines -88 to -90
Copy link
Copy Markdown
Member

@Lms24 Lms24 Mar 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

m: Was debug.info unused?

Copy link
Copy Markdown
Member Author

@logaretm logaretm Mar 10, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Weirdly not, because it was never exported unlike it's friends, so no module had a way of using it.

Happy to put it back and slap an ignore on it.

Copy link
Copy Markdown
Member

@Lms24 Lms24 Mar 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If it's unused let's remove it :) I was just surprised


function warn(...args: Parameters<typeof console.warn>): void {
_maybeLog('warn', ...args);
}
Expand Down