You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
tools: replace deprecated ESLint globals value 'readable' with 'readonly' for EventSource #64757
Not a runtime bug, a leftover deprecated value in the lint config.
In eslint.config.mjs, inside the general globals block, EventSource is declared with the legacy value 'readable' while every other global in the same object uses 'readonly':
// eslint.config.mjs:131EventSource: 'readable',
It is the only remaining occurrence in the repository:
How often does it reproduce? Does it reproduce on all platforms?
Always, the value is static config.
What is the expected behavior? Why is that the expected behavior?
EventSource: 'readonly', matching the other 40+ entries in the same block.
ESLint keeps 'readable' and 'writeable' working only for historical reasons and documents them as deprecated in favour of 'readonly' and 'writable' (see eslint/eslint#11359, which introduced the current spellings).
What do you see instead?
The deprecated spelling is still used. Behavior is unchanged today, since 'readable' is treated as equivalent to 'readonly', so this is a consistency and future-proofing fix rather than a functional one.
Additional information
Origin of the inconsistency: the old .eslintrc.js used 'readable' for all of these globals. The flat config migration in 7e6d92c ("tools: update ESLint to v9 and use flat config") converted them to 'readonly' but missed this single entry:
Yes @Archkon, happy to. I will open a PR that changes the single line to EventSource: 'readonly', and run make lint-js locally first to confirm nothing else shifts. Thanks for taking a look.
Activity
Archkon commentedon Jul 26, 2026
HoneyTyagii commentedon Jul 26, 2026
Yes @Archkon, happy to. I will open a PR that changes the single line to
EventSource: 'readonly', and runmake lint-jslocally first to confirm nothing else shifts. Thanks for taking a look.tools: use 'readonly' for EventSource global
tools: use 'readonly' for EventSource global
tools: use 'readonly' for EventSource global
tools: use 'readonly' for EventSource global
tools: use 'readonly' for EventSource global
tools: use 'readonly' for EventSource global