-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Base setup #3
Base setup #3
Conversation
Add Function and Tests to Extract Issue Numbers from a String
@@ -3,7 +3,7 @@ import XCTest | |||
#if !canImport(ObjectiveC) | |||
public func allTests() -> [XCTestCaseEntry] { | |||
return [ | |||
testCase(ChangelogProducerTests.allTests), | |||
// testCase(ChangelogProducerTests.allTests), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this code commented out? Do we need it for running the test?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're not keeping the SPM tests up to date for now but it's good to keep it in place for if we ever like to add it back. That's why I simply commented it out for now!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, I only have a few small comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, I only have a few small comments
Co-Authored-By: Kaira <kaira@wetransfer.com>
// ChangelogBuilder.swift | ||
// ChangelogProducerCore | ||
// | ||
// Created by Antoine van der Lee on 16/01/2020. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't the copyright be included in the files?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, I'll add that!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! I just have a generic question.
…logProducer into feature/input-commands
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for addressing the feedback.
First of all, sorry for making this PR so big. Due to Play Day I had build everything at once but a bit too hacky. Therefore, I decided to rewrite it before opening the PR.
It does result in code that looks as good as I think it could. I'm really interested to hear your feedback on it.