Skip to content

Carthage update from master#4379

Merged
mnaamani merged 8 commits intoJoystream:carthagefrom
mnaamani:carthage-update-from-master
Oct 17, 2022
Merged

Carthage update from master#4379
mnaamani merged 8 commits intoJoystream:carthagefrom
mnaamani:carthage-update-from-master

Conversation

@mnaamani
Copy link
Copy Markdown
Member

@mnaamani mnaamani commented Oct 17, 2022

Merging master branch into carthage,
dropping commented out code related to building for raspberry-pi armv7 (32-bit)

┆Issue is synchronized with this Asana task by Unito

@vercel
Copy link
Copy Markdown

vercel bot commented Oct 17, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated
pioneer-testnet ⬜️ Ignored (Inspect) Oct 17, 2022 at 0:52AM (UTC)

Copy link
Copy Markdown
Contributor

@Lezek123 Lezek123 left a comment

Choose a reason for hiding this comment

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

Looks good, however I noticed that we're still using outdated version for some GH actions:
image

Is there a specific reason for this, or were they just overlooked?

@mnaamani
Copy link
Copy Markdown
Member Author

Is there a specific reason for this, or were they just overlooked?

Maybe overlooked, I'll double check.

@Lezek123 Lezek123 self-requested a review October 17, 2022 13:00
@mnaamani mnaamani requested a review from Lezek123 October 17, 2022 13:19
Copy link
Copy Markdown
Contributor

@Lezek123 Lezek123 left a comment

Choose a reason for hiding this comment

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

LGTM if the CI checks pass

@mnaamani mnaamani merged commit bef701b into Joystream:carthage Oct 17, 2022
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