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

Fixes lint for windows; adds test/font/fontutils.js #4543

Merged
merged 1 commit into from
Mar 31, 2014

Conversation

yurydelendik
Copy link
Contributor

No description provided.

@yurydelendik
Copy link
Contributor Author

/botio lint

@pdfjsbot
Copy link

From: Bot.io (Linux)


Received

Command cmd_lint from @yurydelendik received. Current queue size: 0

Live output at: http://107.21.233.14:8877/113023d8b3aa289/output.txt

@pdfjsbot
Copy link

From: Bot.io (Windows)


Received

Command cmd_lint from @yurydelendik received. Current queue size: 0

Live output at: http://107.22.172.223:8877/b3575104aeac1ec/output.txt

@pdfjsbot
Copy link

From: Bot.io (Windows)


Failed

Full output at http://107.22.172.223:8877/b3575104aeac1ec/output.txt

Total script time: 0.64 mins

  • Lint: FAILED

@pdfjsbot
Copy link

From: Bot.io (Linux)


Success

Full output at http://107.21.233.14:8877/113023d8b3aa289/output.txt

Total script time: 0.67 mins

  • Lint: Passed

@yurydelendik
Copy link
Contributor Author

/botio-linux lint

@pdfjsbot
Copy link

From: Bot.io (Linux)


Received

Command cmd_lint from @yurydelendik received. Current queue size: 0

Live output at: http://107.21.233.14:8877/b1a1dd3ad5bc9d6/output.txt

@pdfjsbot
Copy link

From: Bot.io (Linux)


Success

Full output at http://107.21.233.14:8877/b1a1dd3ad5bc9d6/output.txt

Total script time: 0.66 mins

  • Lint: Passed

yurydelendik added a commit that referenced this pull request Mar 31, 2014
Fixes lint for windows; adds test/font/fontutils.js
@yurydelendik yurydelendik merged commit 6cf5455 into mozilla:master Mar 31, 2014
@yurydelendik
Copy link
Contributor Author

Unable to make it work on botio-window due to process.stdout is not blocking on windows, so process.exit does not let output to be flushed.

@yurydelendik yurydelendik deleted the winlint branch March 31, 2014 14:37
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