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
Logging consolidation - Remove window.console and console. calls #9721
changed the title [-]Logging consolidation - Remove `window.console` calls[/-][+]Logging consolidation - Remove `window.console` and `console.` calls[/+]on Nov 4, 2021
Activity
karthiknadig commentedon Jan 27, 2020
Check for
console.logas well.karthiknadig commentedon Jan 27, 2020
Direct uses of
console.console.calls to post messages to the web paneldatascience-ui/react-common/postOffice.ts-console.calls overriden herewebPanel.ts-console.logwebPanelServer.ts-console.logdatascience-ui/react-common/logger.ts-console.logsrc/test/:exitCIAfterTestReporter.ts: for tests (CI/AzDO)utils/ts: for functional testsdebuggerTest.ts: for debugger testsdatascience/: DS testsfeatureDeprecationManager.ts: notify deprecation (console.error) ➡️ can be moved to an output channelsocketServer.ts: error in socket server (console.error) ➡️ can be deleteddebugStreamProvider.ts: DA logging for third-party extension devs (console.error) ➡️ can be moved to a debug output channelLocalDebugClient.ts: debugging error logging (console.error) ➡️ can be moved to a debug output channelintellisenseDocument.ts: DS autocompletion (console.warn)container.ts: for tests (console.warn) ➡️ use a logging function insteadkarthiknadig commentedon Jan 27, 2020
Spoke to the DS team. We should not be replacing
console.loginsideDataScience-ui/react*karthiknadig commentedon Jan 30, 2020
Data science portion of this will be cleaned up via #9832
karthiknadig commentedon Jan 30, 2020
Not making change to anything under
debugAdapter. Since we are going to phase it out.[-]Logging consolidation - Remove `window.console` calls[/-][+]Logging consolidation - Remove `window.console` and `console.` calls[/+]eleanorjboyd commentedon Dec 11, 2024
took a look and these are the only places left with
console.. Any of them worth keeping around or can I put in a PR to switch them all totrace?Anthony Kim (@anthonykim1)
Karthik Nadig (@karthiknadig)
karthiknadig commentedon Dec 17, 2024
These can all be switched to trace.
remove console. in production code (microsoft#24629)