Skip to content
This repository has been archived by the owner on Oct 23, 2023. It is now read-only.

Patch luxon to fix LavaMoat issue #154

Merged
merged 2 commits into from
Mar 29, 2023
Merged

Patch luxon to fix LavaMoat issue #154

merged 2 commits into from
Mar 29, 2023

Conversation

FrederikBolding
Copy link
Member

@FrederikBolding FrederikBolding commented Mar 29, 2023 •
edited
Loading

Patch luxon since a few of the snaps have it included in their dependencies and luxon breaks under LavaMoat without the patch.

@FrederikBolding FrederikBolding marked this pull request as ready for review March 29, 2023 22:06
bowensanders
bowensanders approved these changes Mar 29, 2023
View reviewed changes
Copy link
Contributor

@bowensanders bowensanders left a comment

Choose a reason for hiding this comment

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

Local manual testing shows this to be working, LGTM

@FrederikBolding FrederikBolding merged commit 021bfda into main Mar 29, 2023
@FrederikBolding FrederikBolding deleted the fb/lavamoat-patches branch March 29, 2023 23:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@bowensanders bowensanders bowensanders approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@FrederikBolding @bowensanders