T3485
In this way, we can use console.log for debug logs -- then quickly test if any debug logs are remaining.
This diff also removes some lingering debug logs from various places in the app.
Details
Details
- Reviewers
emack - Group Reviewers
Restricted Project - Commits
- rABC646b22d10827: [Cashtab] Replace all console.log statements with error or info and remove…
grep -r console.log src/ -- still a few left, but they are removed in D15804
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- cashtab-no-console-logs
- Lint
Lint Errors Severity Location Code Message Error cashtab/src/components/Common/WalletLabel.js:55 ESLINT no-unused-vars - Unit
No Test Coverage - Build Status
Buildable 28163 Build 55871: Build Diff cashtab-tests Build 55870: arc lint + arc unit
Event Timeline
Comment Actions
Build Bitcoin ABC Diffs / Diff Testing (cashtab-tests) failed.
Tail of the build log:
Installing mock-chronik-client dependencies... /work/modules/mock-chronik-client /work/abc-ci-builds/cashtab-tests added 236 packages, and audited 237 packages in 2s 35 packages are looking for funding run `npm fund` for details found 0 vulnerabilities /work/cashtab /work/modules/mock-chronik-client /work/abc-ci-builds/cashtab-tests added 1718 packages, and audited 1719 packages in 31s 262 packages are looking for funding run `npm fund` for details 1 moderate severity vulnerability To address all issues, run: npm audit fix Run `npm audit` for details. > cashtab@2.9.0 build > node scripts/build.js Creating an optimized production build... Failed to compile. [eslint] src/components/Common/WalletLabel.js Line 55:15: 'walletName' is assigned a value but never used no-unused-vars Search for the keywords to learn more about each error. Build cashtab-tests failed with exit code 1
