Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

feat: allow logs to be written to a file by providing the --logging.file argument #4195

Merged
merged 66 commits into from
Jun 27, 2023

Conversation

jeffsmale90
Copy link
Contributor

@jeffsmale90 jeffsmale90 commented Jan 30, 2023

Introducing the ability to specify a log file for Ganache to write its logs to. No more sifting through endless terminal output to find that one critical message you need. Just provide a path to your desired log file with ganache --logging.file <path-to-log-file>, and you're all set! Now you can sit back, relax, and let Ganache do the heavy lifting while you focus on building awesome applications.

@jeffsmale90 jeffsmale90 changed the title feat: Allow logging to file, and viewing of Detached instances logs feat: supply --logging.file argument for Ganache to write logs to a file Feb 22, 2023
@jeffsmale90
Copy link
Contributor Author

Sorry for the force push, but it had to be in order to remove the detached instances logs logic :/

@jeffsmale90 jeffsmale90 marked this pull request as ready for review February 22, 2023 03:59
@jeffsmale90 jeffsmale90 marked this pull request as draft February 23, 2023 01:47
@jeffsmale90 jeffsmale90 marked this pull request as ready for review February 23, 2023 21:30
Copy link
Member

@cds-amal cds-amal left a comment

Choose a reason for hiding this comment

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

Incomplete review, will finish it tomorrow.

src/chains/ethereum/options/src/logging-options.ts Outdated Show resolved Hide resolved
src/packages/cli/src/cli.ts Outdated Show resolved Hide resolved
@jeffsmale90 jeffsmale90 changed the title feat: supply --logging.file argument for Ganache to write logs to a file feat: allow logs to be written to a file by providing the --logging.file argument Mar 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants