Skip to content

Fix for #118 - Needs discussing - #164

Merged
iainbrighton merged 6 commits into
VirtualEngine:splitfrom
kilasuit:split
Oct 25, 2016
Merged

iainbrighton merged 6 commits into
VirtualEngine:splitfrom
kilasuit:split

Conversation

@kilasuit

Copy link
Copy Markdown
Contributor

@iainbrighton - here is the fix for #118 however there are a few points that need discussed possibly better via skype as this is a big change to the module.

  • Tests - I have left them in as they were whilst also adding in my own non-functional tests in as well - check the Private\Tests folder along with the Public\Verb\Tests folders - you'll see what I mean when you have a look at this.
  • Build.psake.ps1 - I have left this alone as I have added PesterTest.ps1 & Min.Tests.ps1 for localised testing which works for the requirements found so far (some errors as can be found in the testresults.xml)

@iainbrighton

Copy link
Copy Markdown
Contributor

Awesome! That's a s*&^ load of functions...

I'm happy to discuss via Skype. My immediate thinking is that this probably wants to go into a separate branch until the tests and build are working... I think I'll need to create a new branch?

@kilasuit

Copy link
Copy Markdown
Contributor Author

Yes if you could create a new branch "split" that would be great and I'll get working on the failing tests too.

@iainbrighton

Copy link
Copy Markdown
Contributor

@kilasuit Branch split should now be available. Can you push to this branch instead? Thanks!

@kilasuit

Copy link
Copy Markdown
Contributor Author

I'm sure that you can reopen and then change the branch it is going to point to

@kilasuit
kilasuit changed the base branch from dev to split October 25, 2016 17:11
@kilasuit

Copy link
Copy Markdown
Contributor Author

Infact I could do it this is now pointing to the split branch 😉

@iainbrighton iainbrighton reopened this Oct 25, 2016
@iainbrighton
iainbrighton merged commit 187a999 into VirtualEngine:split Oct 25, 2016
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