-
-
Notifications
You must be signed in to change notification settings - Fork 148
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
CI script to run unit tests #98
Conversation
Create unit-test ci.yml
In the scheme their was a typo in Maintenance. Their was an e instead of a in 'nance'
Ohh no it’s for #53 since it wouldn’t build the CI for the unit test.Sent from my iPhoneOn Oct 8, 2023, at 3:19 PM, Mikaela Caron ***@***.***> wrote:
Does this replace #95 ? if so close that PR, and be sure to update the description and title properly for this PR @am2089
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
@am2089 I'm referencing another PR, because you created two now, does this replace your first PR? |
Yes this replaces the first oneSent from my iPhoneOn Oct 8, 2023, at 3:42 PM, Mikaela Caron ***@***.***> wrote:
@am2089 I'm referencing another PR, because you created two now, does this replace your first PR?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
@am2089 it looks like this still isn't running, has it worked on your fork repo? Also I think considering it hasn't worked, we could instead look at using fastlane for this, you're welcome to try to tackle this, issue #54 is working on something similar. If you're not familiar or don't want to work on this issue, no worries, go to the issue and un-assign yourself |
What it Does
CI script to run unit test
How I Tested
Can't be tested until accepted by @mikaelacaron