Skip to content

Commit

Permalink
check
Browse files Browse the repository at this point in the history
  • Loading branch information
chrismccord committed Sep 6, 2024
1 parent c80568c commit e43a339
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions lib/flame/code_sync.ex
Original file line number Diff line number Diff line change
Expand Up @@ -188,7 +188,7 @@ defmodule FLAME.CodeSync do
log_verbose("packaged size: #{File.stat!(out_path).size / (1024 * 1024)}mb")
end

File.stream!(out_path, [], code.chunk_size)
[File.stream!(out_path, [], code.chunk_size) |> Enum.to_list() |> IO.iodata_to_binary()]
end

%PackagedStream{
Expand Down Expand Up @@ -260,7 +260,7 @@ defmodule FLAME.CodeSync do
end

def rm_packaged_stream(%PackagedStream{} = pkg) do

Check warning on line 262 in lib/flame/code_sync.ex

View workflow job for this annotation

GitHub Actions / main (1.15.7, 26.1.2, true)

variable "pkg" is unused (if the variable is not meant to be used, prefix it with an underscore)
if pkg.stream, do: File.rm(pkg.stream.path)
# if pkg.stream, do: File.rm(pkg.stream.path)
:ok
end

Expand Down
2 changes: 1 addition & 1 deletion test/code_sync_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ defmodule FLAME.CodeSyncTest do
|> CodeSync.compute_copy_paths()

assert %FLAME.CodeSync.PackagedStream{} = pkg = CodeSync.package_to_stream(code)
assert File.exists?(pkg.stream.path)
# assert File.exists?(pkg.stream.path)

# cheap way to ensure apps are started on extract. Note async: false is required
Application.stop(:logger)
Expand Down

0 comments on commit e43a339

Please sign in to comment.