Skip to content

Feature/run legacy tests in build#13

Merged
ThatRendle merged 5 commits into
VisualReCode:developfrom
TheRubble:feature/run-legacy-tests-in-build
Apr 27, 2021
Merged

Feature/run legacy tests in build#13
ThatRendle merged 5 commits into
VisualReCode:developfrom
TheRubble:feature/run-legacy-tests-in-build

Conversation

@TheRubble

Copy link
Copy Markdown
Contributor

I've not removed the net5 build, I wasn't sure how you'd feel about that. I've also not changed the build that runs on main as I thought I'd get your feedback.

This now runs the legacy unit tests as part of the PR onto develop.

@ThatRendle ThatRendle merged commit 67d3d97 into VisualReCode:develop Apr 27, 2021
@ThatRendle

Copy link
Copy Markdown
Contributor

LGTM. Thanks!

ThatRendle added a commit that referenced this pull request Jul 19, 2021
* Original package has been deprecated this is the replacement. (#9)

Co-authored-by: TheRubble <therubble@hotmail.com>

* Allow the client to validate the remote certificate, useful in development and self certificate scenarios (#8)

Co-authored-by: TheRubble <therubble@hotmail.com>
Co-authored-by: Mark Rendle <mark@rendlelabs.com>

* Adding basic docs for MVC Fullframework. (#7)

Co-authored-by: TheRubble <therubble@hotmail.com>

* Using statement is incorrect, prevents the build from working. (#11)

* Feature/improving test coverage (#12)

* CookieTests

* Added basic tests before fixing some bugs.

Co-authored-by: TheRubble <therubble@hotmail.com>

* Feature/run legacy tests in build (#13)

* Adding console runner and changes to the build.

* Parallel the build and add the test runner.

* Switched to release.

* Testing the build.

* Reverting test change.

Co-authored-by: TheRubble <therubble@hotmail.com>

Co-authored-by: TheRubble <482262+TheRubble@users.noreply.github.com>
Co-authored-by: TheRubble <therubble@hotmail.com>
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