Skip to content

spell checking my-first-couchdb-plugin's README#185

Closed
marianoguerra wants to merge 3 commits intoapache:masterfrom
marianoguerra:my-first-couch-plugin-readme-typos
Closed

spell checking my-first-couchdb-plugin's README#185
marianoguerra wants to merge 3 commits intoapache:masterfrom
marianoguerra:my-first-couch-plugin-readme-typos

Conversation

@marianoguerra
Copy link
Copy Markdown
Contributor

this pull request only fix spelling mistakes, but there's a part I would like to improve also

the opening note of the plugin README says:

"NOTE: This is incomplete, barely tested, works only with the 1867-feature-plugin branch of Apache CouchDB and expects that you understand some Erlang. This is mostly for early review, but if you are daring, you can learn someting already :)"

the branch name may be outdated since this works on 1.5.0, also the incomplete, barely tested part seems outdated, maybe "this feature is considered alpha/beta/subject to change"?

can you recommend a better warning note?

@andywenk
Copy link
Copy Markdown
Member

You are right about the 1.5 branch so feel free to change this sentence also. "this feature is considered alpha/beta/subject to change" sounds good to me. Would you update the PR?

Thanks :)

@marianoguerra
Copy link
Copy Markdown
Contributor Author

@andywenk updated opening note and git clone url to point to this repo since this works on main couchdb

@asfgit asfgit closed this in e07cc82 Mar 17, 2014
cararemixed pushed a commit to cararemixed/couchdb that referenced this pull request Dec 9, 2014
lag-linaro pushed a commit to lag-linaro/couchdb that referenced this pull request Oct 25, 2018
Switch retest dep to upstream (dizzyd/retest.git)
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.

2 participants