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

Add doc #9849

Merged
merged 8 commits into from Jan 13, 2020
Merged

Add doc #9849

merged 8 commits into from Jan 13, 2020

Conversation

@hkollmann
Copy link
Member

hkollmann commented Jan 11, 2020

for

  • testing, solves #9823
  • migration
hkollmann added 3 commits Jan 9, 2020
@pullapprove pullapprove bot removed the request for review from level420 Jan 11, 2020
docs/compiler/cli/testing.md Outdated Show resolved Hide resolved
docs/compiler/cli/testing.md Outdated Show resolved Hide resolved
@pullapprove pullapprove bot requested a review from zaucker Jan 11, 2020
hkollmann added 2 commits Jan 12, 2020
@@ -1,4 +1,4 @@
### CLI Commands
# CLI Commands

The qooxdoo CLI commands allow to conveniently create, modify, compile, testand publish

This comment has been minimized.

Copy link
@derrell

derrell Jan 13, 2020

Member

testand -> test, and

This comment has been minimized.

Copy link
@hkollmann

hkollmann Jan 13, 2020

Author Member

Done

@@ -1,4 +1,4 @@
### CLI Commands
# CLI Commands

The qooxdoo CLI commands allow to conveniently create, modify, compile, testand publish
and publish your qooxdoo project. To see the main available commands, type

This comment has been minimized.

Copy link
@derrell

derrell Jan 13, 2020

Member

delete duplicated and publish

This comment has been minimized.

Copy link
@hkollmann

hkollmann Jan 13, 2020

Author Member

Done

# Testing of your application

Testing of your qooxdoo app can be done by the qooxdoo cli command `qx test`.
For this purpose you need to install one of the provide test plugins:

This comment has been minimized.

Copy link
@derrell

derrell Jan 13, 2020

Member

provide -> provided

docs/compiler/cli/testing.md Show resolved Hide resolved
${preBootJs}
<script type="text/javascript" src="${appPath}boot.js"></script>
```
- find out which libraries are used in the old application. For this have a look into `compile.json` libraries section. Add all libraries found here with

This comment has been minimized.

Copy link
@derrell

derrell Jan 13, 2020

Member

If that's libraries used in the old application, shouldn't it say to look into config.json rather than compile.json?

This comment has been minimized.

Copy link
@hkollmann

hkollmann Jan 13, 2020

Author Member

Changed

```
Hopefully all needed libraries are converted.

- find used theme in old/compile.json and set this in new/compile.js

This comment has been minimized.

Copy link
@derrell

derrell Jan 13, 2020

Member

old/config.json ?

This comment has been minimized.

Copy link
@hkollmann

hkollmann Jan 13, 2020

Author Member

changed

This comment has been minimized.

Copy link
@derrell

derrell Jan 13, 2020

Member

Looks like it still says old/compile.json instead of old/config.json (unless github isn't updating something...???)

This comment has been minimized.

Copy link
@derrell

derrell Jan 13, 2020

Member

And new/compile.js should probably be new/compile.json I think.

docs/compiler/migration.md Show resolved Hide resolved
```
Hopefully all needed libraries are converted.

- find used theme in old/compile.json and set this in new/compile.js

This comment has been minimized.

Copy link
@derrell

derrell Jan 13, 2020

Member

Looks like it still says old/compile.json instead of old/config.json (unless github isn't updating something...???)

```
Hopefully all needed libraries are converted.

- find used theme in old/compile.json and set this in new/compile.js

This comment has been minimized.

Copy link
@derrell

derrell Jan 13, 2020

Member

And new/compile.js should probably be new/compile.json I think.

Copy link
Member

derrell left a comment

Thanks

@hkollmann hkollmann merged commit 0c8b7ee into qooxdoo:master Jan 13, 2020
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
pullapprove 1 group approved
Details
@hkollmann hkollmann deleted the hkollmann:add_doc_5 branch Jan 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.