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

Jump to bottom

Logs the mode and description of keploy #75

Open

frey0-0 wants to merge 3 commits into keploy:main from frey0-0:log-keploy-mode

frey0-0 commented Mar 24, 2023

Fixes #390

Adds logs in the constructor of keploy class and mock.ts
Uses mode.GetMode() to get the mode instead of process.env as the mode in the .env can be some other than test, record or off
Add a modeDescription() function to log the description as writing a switch case or if there seemed to reduce the readability of the code

frey0-0 added 2 commits

March 24, 2023 23:27


          feat(mock,keploy): log keploy mode

ba48830

Signed-off-by: Tanmay Bajaj <[email protected]>


          feat(mode logs): log description of the mode using a new function

8d3c2d3

Signed-off-by: Tanmay Bajaj <[email protected]>

Contributor

re-Tick commented Mar 26, 2023

LGTM

re-Tick added Don't Merge Accepted labels


          Merge branch 'keploy:main' into log-keploy-mode

d3b8bdb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Accepted Don't Merge