From 3b93268d4b2ea1e98263b0329a3b21bed5a03ef1 Mon Sep 17 00:00:00 2001 From: Dario Maiocchi Date: Sat, 11 Nov 2017 13:16:03 +0100 Subject: Add more doc about the Makefile workflow(suggested) --- doc/CONTRIBUTING.md | 19 +++++++++++++++---- 1 file changed, 15 insertions(+), 4 deletions(-) (limited to 'doc') diff --git a/doc/CONTRIBUTING.md b/doc/CONTRIBUTING.md index bbdfe7af..6fbcf4c8 100644 --- a/doc/CONTRIBUTING.md +++ b/doc/CONTRIBUTING.md @@ -5,15 +5,26 @@ Look at open issues, especially with label: - [junior job](https://github.com/dmacvicar/terraform-provider-libvirt/issues?q=is%3Aissue+is%3Aopen+label%3A%22junior+job%22), - [help wanted](https://github.com/dmacvicar/terraform-provider-libvirt/issues?q=is%3Aissue+is%3Aopen+label%3A%22help+wanted%22) +### Makefile devel workflow. + +Go in your build dir ```cd $GOPATH/src/github.com/dmacvicar/terraform-provider-libvirt```, +then use + +```console +make test +``` + # Contribute -1. [Build the plugin from source](https://github.com/dmacvicar/terraform-provider-libvirt#building-from-source) -2. If you have issues, check out the [troubleshooting](https://github.com/dmacvicar/terraform-provider-libvirt/blob/master/doc/TROUBLESHOOTING.md) -3. Do your code (reference issue on your pr if you fix them) [Closing issues keywords](https://help.github.com/articles/closing-issues-using-keywords/) -4. Run the acceptance tests. +1. If you have issues, check out the [troubleshooting](https://github.com/dmacvicar/terraform-provider-libvirt/blob/master/doc/TROUBLESHOOTING.md) +2. Do your code (reference issue on your pr if you fix them) [Closing issues keywords](https://help.github.com/articles/closing-issues-using-keywords/) +3. Use the Makefile workflow before submit the PR (building, tests). +4. Test your code by running the acceptance tests ```make test``` +5. Create a PR ## Writing acceptance tests. +Look at https://github.com/hashicorp/terraform/blob/master/.github/CONTRIBUTING.md#running-an-acceptance-test ## Running acceptance tests -- cgit v1.2.3