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

(NFC) Move test docs to civicrm-dev-docs #11213

Merged
merged 2 commits into from
Nov 2, 2017
Merged

Conversation

totten
Copy link
Member

@totten totten commented Oct 30, 2017

Overview

This PR removes some test documentation from civicrm-core.git to civicrm-dev-docs.git. However, to preserve discoverability, it provides links to (future/upcoming) dev-docs URL.

NOTE: This depends on civicrm/civicrm-dev-docs#415

Before

The folders tests and tests/qunit have README files with useful docs..

After

The folders tests and tests/qunit have README files with stubs.

As part of civicrm/civicrm-dev-docs#415 , the dev
docs will assimilate and organize more of the instructions about testing.
As part of civicrm/civicrm-dev-docs#415 , the dev
docs will assimilate and organize more of the instructions about testing.
@seamuslee001
Copy link
Contributor

I agree with this and makes sense, Dev Docs is more consolidated and better for this

@seancolsen
Copy link
Contributor

Now that civicrm/civicrm-dev-docs#415 has been merged, it looks like this PR can be merged. Looks good to me too. Ping @totten @eileenmcnaughton

@totten
Copy link
Member Author

totten commented Nov 2, 2017

Thank you @seamuslee001 @seanmadsen

@totten totten merged commit 05a1c19 into civicrm:master Nov 2, 2017
@totten totten deleted the master-test-doc branch November 2, 2017 23:41
sluc23 pushed a commit to ixiam/civicrm-core that referenced this pull request Jan 10, 2018
(NFC) Move test docs to civicrm-dev-docs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants