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

CI / Guthub actions v2 #66

Merged
merged 18 commits into from
Aug 12, 2024
Merged

CI / Guthub actions v2 #66

merged 18 commits into from
Aug 12, 2024

Conversation

Frank1234
Copy link
Collaborator

@Frank1234 Frank1234 commented Jul 31, 2024

Why is this important?

We had a two-headed CI setup, partly fastlane, partly github actions. It increased complexity * 2. Also, signing key usage was not included, we've added it here.

In this ticket

Example actions that ran because of this PR:

@Frank1234 Frank1234 requested review from ninovanhooff, sebaslogen and Techwolf12 and removed request for sebaslogen, Techwolf12 and ninovanhooff July 31, 2024 07:18
@Frank1234 Frank1234 added the changes requested This doesn't seem right label Jul 31, 2024
@Frank1234 Frank1234 requested review from ninovanhooff, sebaslogen and Techwolf12 and removed request for ninovanhooff July 31, 2024 08:45
@Frank1234 Frank1234 removed the changes requested This doesn't seem right label Jul 31, 2024
Techwolf12
Techwolf12 previously approved these changes Jul 31, 2024
ninovanhooff
ninovanhooff previously approved these changes Aug 9, 2024
Copy link
Collaborator

@ninovanhooff ninovanhooff left a comment

Choose a reason for hiding this comment

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

This all looks good, thanks!

None of my comments are blocking, so I'll approve.

But please do consider them :-)


- First create your own keystore and store it in 1pw. _You do not add this keystore to the repo_ (
like we did in this template as an example).
- add a KEYSTORE_BASE_64 [new github repository secret](./settings/secrets/actions/new) that is the
Copy link
Collaborator

Choose a reason for hiding this comment

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

After merging this PR, please trigger ask dependabot build by choosing any open dependabot PR and reacting with @dependabot recreate

If that fails, the changes you made in the secrets need to be reflected into the dependabot secrets; and that should be documented here

Copy link
Collaborator

Choose a reason for hiding this comment

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

@Frank1234 Frank1234 merged commit 98bdacf into main Aug 12, 2024
5 checks passed
@Frank1234 Frank1234 deleted the github-actions-only branch August 12, 2024 09:18
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