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

Windows Loader written in Batch #234

Closed
wants to merge 2 commits into from
Closed

Conversation

Reinboom
Copy link
Contributor

This provides a loader similar in function to the Ruby and Python loader, but setup for windows.

After adding phantomjs to the windows Path variable as well as casperjs/batchbin to the windows Path variable, this should work as it would on OSX. "casperjs "

Although a lot of our web engineers use OS X, the vast majority of our DevQA prefer Windows, so this is vital for enterprise inclusion.

@n1k0
Copy link
Member

n1k0 commented Sep 12, 2012

Wow, thank you! I'll be merging this after some testing as soon as I'll get a windows vm running :)

@AndiDog
Copy link

AndiDog commented Oct 23, 2012

I can confirm this is working, should be better than my simple script because it also handles phantomjs arguments. @n1k0: Please pull this one, I will close my request.

n1k0 added a commit that referenced this pull request Oct 23, 2012
@n1k0
Copy link
Member

n1k0 commented Oct 23, 2012

Merged, thanks for contributing.

PS: dunno why github hasn't closed this one automatically.

@n1k0 n1k0 closed this Oct 23, 2012
@n1k0
Copy link
Member

n1k0 commented Oct 23, 2012

@AndiDog @Reinboom if you have 5 minutes, could you just describe basique installation (if any) & usage of this new tool? Write it here in markdown, I'll add it to the docs myself.

@Reinboom
Copy link
Contributor Author

Phantomjs installation additions

  • Windows users: Append ";C:\phantomjs" to your PATH environment variable. Modify this path appropriately if you installed PhantomJS to a different location.

Casperjs installation additions

  • Windows users: Append ";C:\casperjs\batchbin" to your PATH environment variable. Modify this path appropriately if you installed CasperJS to a different location.

n1k0 added a commit that referenced this pull request Oct 23, 2012
@n1k0
Copy link
Member

n1k0 commented Oct 23, 2012

@Reinboom: thanks!

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.

3 participants