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

[chore] activate github action #37

Open
wants to merge 28 commits into
base: main
Choose a base branch
from
Open

Conversation

dayo2n
Copy link
Collaborator

@dayo2n dayo2n commented Mar 10, 2024

📝 Work Done

swift.yml 파일 추가

💬 References

Github docs

@dayo2n dayo2n added ✨ feat New feature or request 🧹 chore 잡다한 일 labels Mar 10, 2024
@dayo2n dayo2n self-assigned this Mar 10, 2024
@dayo2n
Copy link
Collaborator Author

dayo2n commented Mar 10, 2024

🚨 Never Merge 🚨

@JeaSungLEE
Copy link

JeaSungLEE commented Mar 14, 2024

@dayo2n 깃헙 추천레포로 떠서 구경왔다가 커밋에서 빡침이 느껴져서 코멘트 남깁니다.

먼저 커밋들을 보니까 iOS17과 swfit 5.9의 문법 등등을 제거한 흔적이 보이는데. 엑스코드 커맨드 라인툴이 이전 버전으로 지정 되어있어서 그런것 같아요.
그래서 동작했던 github action 로그를 보니까
image
이미지가 macos 12로 지정되어있더라구요. 그래서 해당 이미지에 대한 정보를 보면 아래 링크와 같이 xcode14가 기본값으로 지정이 되어있습니다.
https://github.com/actions/runner-images/blob/macOS-12/20240218.1/images/macos/macos-12-Readme.md#xcode
찾긴 어렵지만 로그에서도 14.2로 지정되어있는걸 볼수 있었습니다.
image

그래서 먼저 macos버전을 올리는 작업을 하시면서 xcode15가 되도록 하면 iOS17을 타겟으로 하거나 iphone15를 시뮬레이터로 적용하고 #preview와 같은 스위프트 매크로 등등을 사용하실수 있을겁니다.

최신 맥으로 타겟을 하려면
깃헙 워크플로우에 런 macos-latest를 명시적으로 변경하시면 될것 같습니다.
runs-on: macos-14
https://github.com/actions/runner-images/blob/main/images/macos/macos-14-Readme.md

기본값이 15.0.1이여서 macos만 올려도 큰 문제는 없을것 같습니다.

추가로 run하기전에 현재 xcode 버전을 출력해보도록 하면 더 명확해질 것 같습니다.

이 문제부터 해결한다음 다음 발생하는 문제를 한번더 확인해 봐야 할 것 같습니다.

@dayo2n
Copy link
Collaborator Author

dayo2n commented Mar 14, 2024

@JeaSungLEE 놀랍게도 macos 버전만 명시해 주니 바로 통과되었어요!
Xcode 버전에 맞추어 시뮬레이터 버전만 맞춰주면 된다고 생각하고 macos 버전을 의심해보지 못했네요
어쩐지 5.9 문법에서 에러가 발생했었는데 짚고 넘어가지 않았다니...😭
제 커밋..ㅋㅋ을 보고 지나치지 않고 도움을 주셔서 정말 감사합니다 재르시! 🙇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧹 chore 잡다한 일 ✨ feat New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants