-
Notifications
You must be signed in to change notification settings - Fork 30k
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
doc: note about Android support #6040
Conversation
f5d36fc
to
7a2c3bd
Compare
### Android / Android-based devices (e.g., Firefox OS) | ||
|
||
Although these instructions for building on Android are provided, please note | ||
that Android is not a supported platform at this time. Patches to improve |
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.
This is a small nit, but perhaps it should say is not an officially supported platform
?
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.
@mscdex I agree with your nit. Updating now...
LGTM |
1 similar comment
LGTM |
LGTM |
PR-URL: nodejs#6040 Refs: nodejs#6035 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Brian White <mscdex@mscdex.net>
Landed in 0f3c3b4 |
Pull Request check-list
Affected core subsystem(s)
doc build arm
Description of change
Android is not supported at this time, so let's make sure people know that before they get too far.
Refs: #6035