Skip to content
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

Added tvOS as Target #217

Merged
merged 1 commit into from
Dec 15, 2022
Merged

Added tvOS as Target #217

merged 1 commit into from
Dec 15, 2022

Conversation

karm435
Copy link

@karm435 karm435 commented Dec 7, 2022

Added tvOS as Target in package.swift

Related to issue: #216

@karm435 karm435 requested a review from AvdLee as a code owner December 7, 2022 22:57
@SwiftLeeBot
Copy link
Collaborator

Fails
🚫

danger-swift failed.

Log

/var/folders/11/nh0v1jld7zd7b9zqm1774gtm0000gn/T/_tmp_dangerfile.swift:1:8: error: no such module 'Danger'
import Danger
       ^
�[31mERROR: Dangerfile eval failed at Dangerfile.swift
�[0;0m�[31mERROR: Could not get the results JSON file at /var/folders/11/nh0v1jld7zd7b9zqm1774gtm0000gn/T/danger-response.json
�[0;0m

Generated by 🚫 dangerJS against b7b2707

@AvdLee
Copy link
Owner

AvdLee commented Dec 13, 2022

@karm435 do the tests succeed locally? CI is currently broken, so I need some time to fix that

@karm435
Copy link
Author

karm435 commented Dec 13, 2022 via email

@karm435
Copy link
Author

karm435 commented Dec 13, 2022

I ran the tests locally and looks like that are fine.
image

@AvdLee
Copy link
Owner

AvdLee commented Dec 14, 2022

@karm435 they completed, but did the succeed as well?

@karm435
Copy link
Author

karm435 commented Dec 14, 2022

@karm435 they completed, but did the succeed as well?

They are completed with Green color

image

@AvdLee AvdLee merged commit 7aa4da1 into AvdLee:master Dec 15, 2022
@AvdLee
Copy link
Owner

AvdLee commented Dec 15, 2022

Thanks for your contribution @karm435

@karm435
Copy link
Author

karm435 commented Dec 15, 2022

You are welcome. I will see what else I can contribute to as I develop my app.

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.

4 participants