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

Add apple silicon targets #118

Merged

Conversation

ychescale9
Copy link
Contributor

Fixes or Changes Proposed

PR Checklist

  • I have added a CHANGELOG.md entry for any noteable changes or fixes.
  • I have added test coverage for any new behavior or bug fixes.

@benasher44
Copy link
Owner

Ignore the danger check failing. It only works for contributors that can access this repo's default github token

@benasher44
Copy link
Owner

Might work if I manually rerun it though 🤞

@benasher44
Copy link
Owner

Shoot okay. I'll need to make some CI updates this weekend—apologies

@benasher44
Copy link
Owner

@ychescale9 can you please update with master? CI no longer requires the GITHUB_TOKEN secret to run, so you should able to get a clean CI run (though upgrading ktlint may require running ktlintformat, in case there are new rules that need to be applied).

@ychescale9 ychescale9 force-pushed the yc/add-apple-silicon-targets branch from ee6a8ae to f3db4c0 Compare September 6, 2021 15:40
@ychescale9
Copy link
Contributor Author

Rebased.

@benasher44
Copy link
Owner

@ychescale9 landed some more fixes. Try rebasing one more time 🤞

@ychescale9 ychescale9 force-pushed the yc/add-apple-silicon-targets branch from f3db4c0 to b3d2fd6 Compare September 7, 2021 00:03
@ychescale9
Copy link
Contributor Author

Done :)

@benasher44
Copy link
Owner

@ychescale9 okay one more time. Landed another fix

@ychescale9 ychescale9 force-pushed the yc/add-apple-silicon-targets branch from b3d2fd6 to 95522dc Compare September 7, 2021 04:28
@ychescale9
Copy link
Contributor Author

Done

@benasher44 benasher44 merged commit 653dff8 into benasher44:master Sep 7, 2021
@ychescale9 ychescale9 deleted the yc/add-apple-silicon-targets branch September 7, 2021 22:04
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