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

build: run-app.sh also logs to stdout #1260

Closed

Conversation

rzr
Copy link
Contributor

@rzr rzr commented Aug 7, 2018

This can be useful for journalctl or docker etc

Change-Id: I213ab5b4a9a23b925de53aa0758cda33fb452be7
Signed-off-by: Philippe Coval [email protected]

rzr added a commit to TizenTeam/gateway that referenced this pull request Aug 7, 2018
This can be useful for journalctl or docker etc

Change-Id: I213ab5b4a9a23b925de53aa0758cda33fb452be7
Bug: WebThingsIO#1260
Signed-off-by: Philippe Coval <[email protected]>
@rzr rzr force-pushed the sandbox/rzr/review/fix/master branch from b91a9cd to 162f7d7 Compare August 7, 2018 22:25
rzr added a commit to TizenTeam/gateway that referenced this pull request Aug 7, 2018
This can be useful for journalctl or docker etc

Change-Id: I213ab5b4a9a23b925de53aa0758cda33fb452be7
Bug: WebThingsIO#1260
Signed-off-by: Philippe Coval <[email protected]>
@codecov-io
Copy link

codecov-io commented Aug 7, 2018

Codecov Report

Merging #1260 into master will decrease coverage by 0.08%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1260      +/-   ##
==========================================
- Coverage   75.33%   75.25%   -0.09%     
==========================================
  Files         123      123              
  Lines        5916     5916              
  Branches      824      824              
==========================================
- Hits         4457     4452       -5     
- Misses       1287     1291       +4     
- Partials      172      173       +1
Impacted Files Coverage Δ
src/db.js 80.34% <0%> (-1.74%) ⬇️
src/models/thing.js 63.63% <0%> (-1.66%) ⬇️
src/models/things.js 97.4% <0%> (-1.3%) ⬇️
src/controllers/things_controller.js 79.82% <0%> (+0.42%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f7333ac...a79a097. Read the comment docs.

This can be useful for journalctl or docker etc

Change-Id: I213ab5b4a9a23b925de53aa0758cda33fb452be7
Bug: WebThingsIO#1260
Signed-off-by: Philippe Coval <[email protected]>
@rzr rzr force-pushed the sandbox/rzr/review/fix/master branch from 162f7d7 to a79a097 Compare August 10, 2018 16:13
hobinjk pushed a commit that referenced this pull request Aug 10, 2018
This can be useful for journalctl or docker etc

Change-Id: I213ab5b4a9a23b925de53aa0758cda33fb452be7
Bug: #1260
Signed-off-by: Philippe Coval <[email protected]>
@rzr
Copy link
Contributor Author

rzr commented Aug 10, 2018

Merged in other PR:

#1264

@rzr rzr closed this Aug 10, 2018
rzr added a commit to TizenTeam/gateway that referenced this pull request Aug 21, 2018
This can be useful for journalctl or docker etc

Change-Id: I213ab5b4a9a23b925de53aa0758cda33fb452be7
Bug: WebThingsIO#1260
Signed-off-by: Philippe Coval <[email protected]>
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.

2 participants