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

chore: Add an entry for zip_fix engine #277

Merged
merged 2 commits into from
Apr 12, 2023
Merged

chore: Add an entry for zip_fix engine #277

merged 2 commits into from
Apr 12, 2023

Conversation

eseidel
Copy link
Contributor

@eseidel eseidel commented Apr 11, 2023

This adds an entry to artifact_proxy for my in-progress fix to the engine:
https://github.com/shorebirdtech/engine/tree/zip_fix

@eseidel eseidel changed the title Add an entry for zip_fix engine chore: Add an entry for zip_fix engine Apr 11, 2023
'f68335f3f6afdb1595420b1c3b5a16c8da75a1cf': _EngineMapping3_7(
shorebirdStorageBucket: 'download-dev.shorebird.dev',
'adb70a20d4718b5ce60cdd99ad81d8de54afcb35': _EngineMapping3_7(
// shorebirdStorageBucket: 'download-dev.shorebird.dev',
Copy link
Contributor

@felangel felangel Apr 11, 2023

Choose a reason for hiding this comment

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

You want this using the prod bucket?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Up to you just wanted to make sure this was intentional

Copy link
Contributor

@felangel felangel left a comment

Choose a reason for hiding this comment

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

I’m confused about the unused variable warning haha but I’ll take a closer look when I’m at my computer.

@eseidel
Copy link
Contributor Author

eseidel commented Apr 12, 2023

I’m confused about the unused variable warning haha but I’ll take a closer look when I’m at my computer.

That's working around dart-lang/sdk#52016 🤣

@eseidel eseidel merged commit 2af6bbe into main Apr 12, 2023
@eseidel eseidel deleted the zip_proxy branch April 12, 2023 18:21
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