-
Notifications
You must be signed in to change notification settings - Fork 14
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
docs: include noscript installation #191
Conversation
@tisonkun I'm not sure |
@zyy17 fair enough. What about we switch the order? Explain how to install without |
Look good to me. |
@zyy17 Sorry I miss this reply. Patch updated. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #191 +/- ##
========================================
Coverage 37.76% 37.76%
========================================
Files 16 16
Lines 1300 1300
========================================
Hits 491 491
Misses 704 704
Partials 105 105 ☔ View full report in Codecov by Sentry. |
* docs: recommend noscript installation * Update README.md
go install
is trustworthy.More and more users would doubt if an install script from the Internet is safe ..