-
Notifications
You must be signed in to change notification settings - Fork 341
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
some README updates #483
some README updates #483
Conversation
- badges - more emphasis on setup-cml than docker
d07798b
to
add66eb
Compare
- also prevents pre-commit hook from incorrectly reflowing
btw |
930f908
to
89a12f1
Compare
89a12f1
to
7ee37df
Compare
Yes, it's a known issue on our current testing setup (#183). We should either skip them for external pull requests or just run them on every pull request after asking for human approval through GitHub Environments. Anyhow, maybe we should separate unit tests from acceptance tests for a better contribution experience. |
This comment has been minimized.
This comment has been minimized.
[docs about configuring CML with GitLab](https://github.com/iterative/cml/wiki/CML-with-GitLab). | ||
- **Bitbucket Cloud users**: Please see our | ||
[docs on CML with Bitbucket Cloud](https://github.com/iterative/cml/wiki/CML-with-Bitbucket-Cloud). | ||
_Bitbucket Server support estimated to arrive by May 2021._ |
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 changed this from "Jan 2021" to "May 2021" but idk if that's still too ambitious :)
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 think that we actually support BitBucket to some extent.
@casperdcl, is this ready for merging? |
* some README updates - badges - more emphasis on setup-cml than docker * fix yaml -> bash - also prevents pre-commit hook from incorrectly reflowing * readme: update badge label * em-dash * readme: more copyedits * docs: more readme clarifications * better emoji support
Fixes #482