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

build: change to get api credentials from xcconfig file #46

Merged
merged 1 commit into from
Oct 2, 2023
Merged

build: change to get api credentials from xcconfig file #46

merged 1 commit into from
Oct 2, 2023

Conversation

cre8ivejp
Copy link
Member

@cre8ivejp cre8ivejp commented Sep 25, 2023 •
edited
Loading

To avoid accidentally pushing the dev environment credentials, I changed the example apps to get the API credentials from the environment.xcconfig.

Things done

  • Change to get the API credentials from the xcconfig file
  • Moved the files from scripts to hack directory
  • Added option to cancel the current workflow when a new one is created
  • Updated README explaining how to set up the credentials

@cre8ivejp cre8ivejp force-pushed the chore-api-setup branch 18 times, most recently from 30361cb to 5b01b8a Compare September 25, 2023 12:32
@cre8ivejp cre8ivejp changed the title build: change to get api credentials from setup file build: change to get api credentials from xxconfig file Sep 25, 2023
@cre8ivejp cre8ivejp changed the title build: change to get api credentials from xxconfig file build: change to get api credentials from xcconfig file Sep 25, 2023
@cre8ivejp cre8ivejp force-pushed the chore-api-setup branch 2 times, most recently from c238492 to eebce32 Compare September 25, 2023 12:53
@cre8ivejp cre8ivejp merged commit cb76294 into main Oct 2, 2023
@cre8ivejp cre8ivejp deleted the chore-api-setup branch October 2, 2023 04:54
@bucketeer-bot bucketeer-bot mentioned this pull request Sep 27, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@cre8ivejp