Skip to content

Update static.zip, automate its release process #60

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

Merged
merged 2 commits into from
Jul 19, 2020

Conversation

MaxDesiatov
Copy link
Collaborator

@MaxDesiatov MaxDesiatov commented Jul 19, 2020

dev.js entrypoint has been updated to include the recent JavaScriptKit runtime fix: swiftwasm/JavaScriptKit#19

I've also added a new subcommand to carton-release, which is now able to automaticallly create a new static.zip and record updated hashes in the source code. It still doesn't upload the archive automatically to previous release assets, but I wonder if that should be done manually anyway, at least until we have some kind of tests that verify the whole process end-to-end.

Additionally, since the new runtime is not compatible with the old Swift parts of JavaScriptKit, carton dev now checks the revision of JavaScriptKit that projects have specified in their Package.swift. It doesn't block the build process, but I hope it gives enough warning about the incompatibility.

@MaxDesiatov MaxDesiatov added the dependencies Updates to the project dependencies label Jul 19, 2020
@@ -9,7 +9,7 @@ let package = Package(
.executable(name: "TestApp", targets: ["TestApp"]),
],
dependencies: [
.package(url: "https://github.com/kateinoigakukun/JavaScriptKit", .revision("85b8617")),
.package(url: "https://github.com/kateinoigakukun/JavaScriptKit", .revision("c90e82f")),
],
targets: [
// Targets are the basic building blocks of a package. A target can define a module or a test suite.
Copy link
Contributor

Choose a reason for hiding this comment

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

  • 🚫 Line should be 100 characters or less: currently 104 characters (line_length)

@ie-ahm-robox
Copy link
Contributor

Fails
🚫

TestApp/Package.swift#L16 - Line should be 100 characters or less: currently 118 characters (line_length)

Generated by 🚫 Danger Swift against 8968d42

@MaxDesiatov MaxDesiatov requested a review from carson-katri July 19, 2020 20:48
@MaxDesiatov MaxDesiatov merged commit 749a17d into main Jul 19, 2020
@MaxDesiatov MaxDesiatov deleted the update-static-archive branch July 19, 2020 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Updates to the project dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants