Skip to content
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.

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

CLI refactoring #1090

Merged
merged 1 commit into from
Jul 21, 2023
Merged

CLI refactoring #1090

merged 1 commit into from
Jul 21, 2023

Conversation

255kb
Copy link
Member

@255kb 255kb commented Jul 21, 2023

  • remove PM2 dependency
  • remove list and stop commands, remove --pname flag
  • make foreground the default and remove the --daemon-flag
  • simplify the dockerize command to only generates a Dockerfile, and enable file downloading
  • remove some dependencies to simplify maintenance (inquirer, etc.) Closes RFC: Remove PM2 dependency from Mockoon CLI #1081

Technical implementation details

Checklist

  • data migration added (@mockoon/commons)
  • data migration automated tests added (@mockoon/commons)
  • CLI automated tests added (@mockoon/cli)
  • desktop automated tests added (@mockoon/desktop)

@255kb 255kb force-pushed the feature/1081-cli-pm2-removal branch from 5fd3cbd to 48b74bc Compare July 21, 2023 07:35
- remove PM2 dependency
- remove list and stop commands, remove `--pname` flag
- make foreground the default and remove the `--daemon-flag`
- simplify the dockerize command to only generates a Dockerfile, and enable file downloading
- remove some dependencies to simplify maintenance (inquirer, etc.)
Closes #1081
@255kb 255kb force-pushed the feature/1081-cli-pm2-removal branch from 48b74bc to afe986a Compare July 21, 2023 08:58
@255kb 255kb merged commit 263716e into release/next Jul 21, 2023
20 checks passed
@255kb 255kb deleted the feature/1081-cli-pm2-removal branch July 21, 2023 09:14
255kb added a commit that referenced this pull request Jul 24, 2023
* Logs standardization (#1064)

- add log transaction option to serverless and desktop
- extract server logging to commons-server
- standardize how server is logging its events
- use same logger (winston) for all the applications
- redact authorization headers when logging to file
Closes #978
Closes #688
Closes #1063

* Add context menu entry to move an environment file (#1065)

Closes #1062

* Review desktop messages and logging after #1063

* Fix documentation generator

* Enable OpenAPI `example` export (#1067)

fixes OpenAPI export crashing with CRUD routes
Closes #352
Closes #1066

* Make `data` helpers compatible with safestring (#1071)

Closes #1069

* Remove reload view and rewrite change detection

use custom rxjs operator that listen to the object changed and some reducer actions that always force the refresh
Also disable template generation on pressing enter if quota reached

* Switch lerna version to fixed

* Add stringify option to oneof helper

* Allow search of hidden routes in the desktop app (#1083)

Automatically uncollapse parent folders of selected route after a search
Closes #960

* Fix for faker methods with numbers in their names (#1084)

* Enable watch mode for libraries

Closes #1076

* Added crudkey property, updated environment and databucket actions (#1082)

* Added crudkey property, updated environment and databucket actions

* Updated crudKey to be a property of RouteResponse
Closes #1041

* Fix package lock and crudKey tooltip wording

* Move crudKey field next to CRUD data bucket selector in desktop app

* CLI refactoring (#1090)

- remove PM2 dependency
- remove list and stop commands, remove `--pname` flag
- make foreground the default and remove the `--daemon-flag`
- simplify the dockerize command to only generates a Dockerfile, and enable file downloading
- remove some dependencies to simplify maintenance (inquirer, etc.)
Closes #1081

* Update repository images

* Update CLI documentation headers and links

* Enable truncation for log menu entries in desktop app

Closes #1085

* Update dependencies

* New prettier format

---------

Co-authored-by: ajatkj <ajatkj@yahoo.co.in>
Co-authored-by: n1ce1041 <70496526+n1ce1041@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant