Skip to content
This repository has been archived by the owner on Jun 8, 2022. It is now read-only.

Vagrant file, ref #59 #61

Merged
merged 3 commits into from
Sep 9, 2013
Merged

Vagrant file, ref #59 #61

merged 3 commits into from
Sep 9, 2013

Conversation

nathany
Copy link
Contributor

@nathany nathany commented Sep 8, 2013

This it what I've come up with a Vagrantfile so far, though it needs boxes for other operating systems.

  • Install Vagrant and VirtualBox
  • Run vagrant up
  • It's now possible to run tests against a Linux box with vagrant ssh linux -c 'go test -v ./...'

* git ignore .vagrant
* fixup example test to use the github path (otherwise need to install as a test dependency)
@nathany
Copy link
Contributor Author

nathany commented Sep 8, 2013

Now there is a BSD box as well. I'm not sure if Windows is doable, but hopefully this helps with testing.

@nathany
Copy link
Contributor Author

nathany commented Sep 8, 2013

@howeyc Do you think this will be helpful for contributors testing changes?

@nathany
Copy link
Contributor Author

nathany commented Sep 9, 2013

I'm adding a contributing file with instructions.

some guidance
@nathany
Copy link
Contributor Author

nathany commented Sep 9, 2013

@howeyc This should be good to merge, though I've only tested it on an OS X host. I'm curious how well it works for you vs. your current methods.

We may want to add something at the top of Contributing for issues and questions, since this file is linked to whenever someone hits "New Issue" on GitHub.

@howeyc howeyc merged commit 07ba8c7 into howeyc:master Sep 9, 2013
@howeyc
Copy link
Owner

howeyc commented Sep 9, 2013

I really like the Contributing file!

I think that Vagrant file may be helpful for those who need Linux/BSD machines to test.

Thanks.

@nathany nathany deleted the vagrant_testing branch September 9, 2013 23:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants