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

Improve documentation for the configure of package.json #8699

Merged

Conversation

zhaomenghuan
Copy link
Contributor

What it does

Improve documentation for the configure of package.json

How to test

Documentation changes only

Review checklist

Reminder for reviewers

Signed-off-by: zhaomenghuan [email protected]

Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

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

@zhaomenghuan please be sure to add a proper sign-off to the commit so we can accept your changes (at the moment the eca check fails due to it).

dev-packages/cli/README.md Outdated Show resolved Hide resolved
@vince-fugnitto vince-fugnitto added documentation issues related to documentation electron issues related to the electron target theia-cli issues related to the theia-cli labels Nov 2, 2020
@zhaomenghuan zhaomenghuan force-pushed the doc/improve-documentation branch 4 times, most recently from fedde48 to b3f3af4 Compare November 3, 2020 04:14
The following commit adds documentation to the `cli`,
to document the electron target configuration including the
possible options such as `disallowReloadKeybinding` and `windowOptions`.

Signed-off-by: zhaomenghuan <[email protected]>

Improve documentation for the configure of package.json

Signed-off-by: zhaomenghuan <[email protected]>
@vince-fugnitto vince-fugnitto force-pushed the doc/improve-documentation branch from b3f3af4 to e53e683 Compare November 3, 2020 14:06
Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

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

Thank you for the changes 👍
I updated the pull-request to squash the commits along with some other minor updates.

@vince-fugnitto vince-fugnitto merged commit 123bf41 into eclipse-theia:master Nov 4, 2020
@zhaomenghuan zhaomenghuan deleted the doc/improve-documentation branch February 21, 2021 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation issues related to documentation electron issues related to the electron target theia-cli issues related to the theia-cli
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants