Skip to content

Build improvements#78

Merged
ErikSchierboom merged 1 commit intoexercism:masterfrom
ErikSchierboom:build-improvements
Aug 14, 2016
Merged

Build improvements#78
ErikSchierboom merged 1 commit intoexercism:masterfrom
ErikSchierboom:build-improvements

Conversation

@ErikSchierboom
Copy link
Copy Markdown
Member

@ErikSchierboom ErikSchierboom commented Aug 10, 2016

  • Use Paket for dependency management
  • Normalize test file names
  • Fix not all tests being run

@kytrinyx
Copy link
Copy Markdown
Member

kytrinyx commented Aug 11, 2016

Is this catching existing test failures?

@ErikSchierboom
Copy link
Copy Markdown
Member Author

ErikSchierboom commented Aug 12, 2016

Yes it is. There is still a bug somewhere, but when it's done it will automatically run all tests using the example code.

@kytrinyx
Copy link
Copy Markdown
Member

kytrinyx commented Aug 13, 2016

OK cool. How do you feel about merging this with a failing build? That always makes me super nervous, but on the other hand it's already broken, this is just finally saying so.

@ErikSchierboom
Copy link
Copy Markdown
Member Author

ErikSchierboom commented Aug 13, 2016

I will only merge this once it's working again. This is basically a work in progress.

@ErikSchierboom
Build improvements
fd5a901 
- Use Paket for dependency management
- Normalize test file names
- Fix not all tests being run
@ErikSchierboom ErikSchierboom merged commit 94680b2 into exercism:master Aug 14, 2016
@ErikSchierboom ErikSchierboom deleted the build-improvements branch August 14, 2016 13:40
@ErikSchierboom ErikSchierboom mentioned this pull request Mar 31, 2021
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ErikSchierboom @kytrinyx