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.

Sign up for GitHub

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

Jump to bottom

Created a profiles package so provisioning profiles only need to be in one place. #794

Merged
merged 2 commits into from
May 3, 2022
Merged

Created a profiles package so provisioning profiles only need to be in one place. #794

merged 2 commits into from
May 3, 2022

Conversation

pmarkowsky
Copy link
Contributor

@pmarkowsky pmarkowsky commented Apr 29, 2022 •
edited
Loading

This PR puts all of the provisioning profiles in a top level profiles package.

⚠️ This will break your build until you copy the provisioningprofile's to the /profiles directory.

@pmarkowsky pmarkowsky requested a review from tburgin April 29, 2022 14:54
@pmarkowsky pmarkowsky self-assigned this Apr 29, 2022
@pmarkowsky pmarkowsky requested a review from mlw April 29, 2022 19:55
russellhancox
russellhancox reviewed Apr 29, 2022
View reviewed changes
Source/santa/BUILD Show resolved Hide resolved
profiles/BUILD Show resolved Hide resolved
@pmarkowsky pmarkowsky merged commit 5caedeb into google:main May 3, 2022
@pmarkowsky pmarkowsky deleted the markowsky/consolidate-dev-profiles branch May 3, 2022 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@russellhancox russellhancox russellhancox approved these changes

@tburgin tburgin Awaiting requested review from tburgin

@mlw mlw Awaiting requested review from mlw

Assignees

@pmarkowsky pmarkowsky

Labels
cleanup
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pmarkowsky @russellhancox