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

Migrate Xcode project to use Folders instead of Groups #101

Merged
merged 1 commit into from
Mar 9, 2025

Conversation

Supereg
Copy link
Member

@Supereg Supereg commented Mar 9, 2025

Migrate Xcode project to use Folders instead of Groups

♻️ Current situation & Problem

This PR migrates the Xcode project file to use Folders instead of the Groups. Refer to Convert groups to folders to learn everything about the conversion and why folders are better and groups. The main benefits: a) we don't incur any merge conflicts as files no longer are tracked in the Xcode project file and b) everything is exactly as on the file system creating less confusion and creating a similar development experience as with Swift Packages.

Overall, this should help especially people new to Xcode to have way less problems start off with iOS development.

Note that all the .license files e.g. in the Resources folder are now also part of the project and distributed with the app which, in my opinion, isn't even a bad side effect and also helps with people getting started with REUSE license files as they are now visible in Xcode and no longer require a separate editor.

⚙️ Release Notes

  • Migrate groups to folders.

📚 Documentation

--

✅ Testing

--

📝 Code of Conduct & Contributing Guidelines

By creating and submitting this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

Copy link

codecov bot commented Mar 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.63%. Comparing base (8d45479) to head (9441b37).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #101   +/-   ##
=======================================
  Coverage   86.63%   86.63%           
=======================================
  Files          21       21           
  Lines         688      688           
=======================================
  Hits          596      596           
  Misses         92       92           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8d45479...9441b37. Read the comment docs.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Member

@PSchmiedmayer PSchmiedmayer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, a great idea! 🚀

@Supereg Supereg merged commit 6d078e3 into main Mar 9, 2025
11 checks passed
@Supereg Supereg deleted the feature/xcode-folders branch March 9, 2025 16:54
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.

2 participants