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

Implement custom Deriveddata path in WorkspaceSettings #650

Merged
merged 5 commits into from
Oct 14, 2021

Conversation

freddi-kit
Copy link
Contributor

@freddi-kit freddi-kit commented Oct 8, 2021

Short description πŸ“

Implemented DerivedData in WorkspaceSettings

Solution πŸ“¦

I added variables for DerivedData customization in WorkspaceSettings.

Implementation πŸ‘©β€πŸ’»πŸ‘¨β€πŸ’»

  • Add derivedDataLocationStyle and derivedDataCustomLocation to represent below code in WorkspaceSettings.xcsettings
    • It is necessary to add new variable likeXCUserData to XcodeProj.swift later becuse normally below settings are set in project.xcworkspace/xcuserdata/{username}.xcuserdatad/WorkspaceSettings.xcsettings
	<key>DerivedDataCustomLocation</key>
	<string>customized-derivedData-path</string>
	<key>DerivedDataLocationStyle</key>
	<string>WorkspaceRelativePath</string>
	<key>DerivedDataLocationStyle</key>
	<string>Default</string>
	<key>DerivedDataCustomLocation</key>
	<string>/Users/---/customized-derivedData-path</string>
	<key>DerivedDataLocationStyle</key>
	<string>AbsolutePath</string>

@codecov-commenter
Copy link

codecov-commenter commented Oct 8, 2021

Codecov Report

Merging #650 (3114b35) into main (78895f1) will increase coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #650      +/-   ##
==========================================
+ Coverage   84.62%   84.66%   +0.04%     
==========================================
  Files         154      154              
  Lines        8850     8878      +28     
==========================================
+ Hits         7489     7517      +28     
  Misses       1361     1361              
Impacted Files Coverage Ξ”
Sources/XcodeProj/Project/WorkspaceSettings.swift 95.65% <100.00%> (+1.90%) ⬆️
...codeProjTests/Project/WorkspaceSettingsTests.swift 100.00% <100.00%> (ΓΈ)

Continue to review full report at Codecov.

Legend - Click here to learn more
Ξ” = absolute <relative> (impact), ΓΈ = not affected, ? = missing data
Powered by Codecov. Last update 78895f1...3114b35. Read the comment docs.

@pepicrft
Copy link
Contributor

@all-contributors add @freddi-kit for code

@allcontributors
Copy link
Contributor

@pepibumur

I've put up a pull request to add @freddi-kit! πŸŽ‰

@pepicrft
Copy link
Contributor

Thanks a lot for contributing this one @freddi-kit

@pepicrft pepicrft merged commit 4978785 into tuist:main Oct 14, 2021
@freddi-kit freddi-kit deleted the dervided-path branch October 21, 2021 00:40
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.

3 participants