-
Notifications
You must be signed in to change notification settings - Fork 81
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 warning message to Windows installation section #1257
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## development #1257 +/- ##
============================================
Coverage 98.00% 98.00%
============================================
Files 89 89
Lines 4506 4506
============================================
Hits 4416 4416
Misses 90 90 |
e46505d
to
00e6692
Compare
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.
lgtm, but let's wait with merge for @ddoktorski
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.
Apart from a small nitpick about rephrasing, would you mind formatting the PR titles (and therefore commit messages in the development branch) to start with a capital letter and be in the imperative mood? Also, it would be great if we could include a 'Closes...' directive in the PR's description.
Co-authored-by: ddoktorski <45050160+ddoktorski@users.noreply.github.com>
Closes #1173
Adds a warning to instructions on how to install starnet.py on Windows