-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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 Policy re. unstable tests to README #782
Conversation
695c728
to
c857f36
Compare
The latest "force push" just now just added |
Why is the other Travis PR test failing? I see this:
|
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.
I suggest that instead of having all this details stated out in one big paragraph, wouldn't it be better if the instructions were listed out in points form?. Reading a huge paragraph is kind of tedious and difficult to track. I have no problem with it though. Just think it would look better that way
c857f36
to
a39d132
Compare
That is an excellent point, you are absolutely right, it was a bit "dense"... how about how it's now - better? |
Yeah, definitely. That looks more like it. |
+1 from me. |
@awasum OK for you?
@nikpawar89 makes sense?
@xurror @maektwain @fynmanoj @thesmallstar @DBryzz @wkk91193 @nemux @nnatarajan FYI!