-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
feat: initial desktop starting environment #103
feat: initial desktop starting environment #103
Conversation
# Conflicts: # src/bin/cli.js
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome 👍
You have committed a submodule desktop
, was that on purpose or?
i dont know why that is present. I thought i deleted it but it persists .i will remove it. @jonaslagoni |
@jonaslagoni ok removed it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keep in mind that with no documentation or tests it's hard for users to use it 🙂
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
@@ -38,7 +39,8 @@ | |||
"commander": "^7.2.0", | |||
"cross-env": "^7.0.3", | |||
"js-yaml": "^3.14.1", | |||
"randexp": "^0.5.3" | |||
"randexp": "^0.5.3", | |||
"run-script": "^0.1.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@NektariosFifes not sure why you use this. But be careful when using dependencies with such a low activitiy.
Easy to introduce vulnerabilities
🎉 This PR is included in version 0.6.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Description
Related issue(s)
Progresses#94