Skip to content
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

adding support document #767

Merged
merged 2 commits into from
Mar 7, 2018
Merged

adding support document #767

merged 2 commits into from
Mar 7, 2018

Conversation

dmcassel
Copy link
Contributor

@dmcassel dmcassel commented Mar 1, 2018

No description provided.

@dmcassel dmcassel requested a review from popzip March 1, 2018 16:01
@marklogic-builder
Copy link

Can one of the admins verify this patch?

1 similar comment
@marklogic-builder
Copy link

Can one of the admins verify this patch?

Copy link
Contributor

@popzip popzip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dmcassel wanted to test what is your thinking on putting as section within readme a la https://github.com/marklogic/java-client-api#support vs as separate doc. I could see it either way.

@dmcassel
Copy link
Contributor Author

dmcassel commented Mar 6, 2018

I interpreted the ticket as saying it should be a separate doc, but that's probably because I like it being separately available. However, looking at the other projects on github.com/marklogic, it is part of the readme. I'll move it.

@dmcassel
Copy link
Contributor Author

dmcassel commented Mar 6, 2018

@popzip I've moved the support statement into the readme

@marklogic-builder
Copy link

Jenkins Build failure

480 tests run, 0 skipped, 2 failed.

@dmcassel dmcassel merged commit 4857076 into develop Mar 7, 2018
@aebadirad aebadirad deleted the support-language branch March 21, 2018 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants