Skip to content

unable to build, tried to improve README#112

Closed
enderversing wants to merge 0 commit intoprocessing:mainfrom
enderversing:main
Closed

unable to build, tried to improve README#112
enderversing wants to merge 0 commit intoprocessing:mainfrom
enderversing:main

Conversation

@enderversing
Copy link
Copy Markdown

@enderversing enderversing commented Apr 14, 2026

Fixes: #111

I tried to make the README.md easier to follow, then I went through each instruction.

error[E0282]: type annotations needed  ] 423/499: gltf-json, wgsl-parse(build), midir,…
  --> /home/username/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/midir-0.10.3/src/backend/webmidi/mod.rs:38:44
   |
38 |             on_ok: Closure::wrap(Box::new(|access| {
   |                                            ^^^^^^
...
41 |                     let access: MidiAccess = access.dyn_into().unwrap();
   |                                              ------ type must be known at this point
   |
help: consider giving this closure parameter an explicit type
   |
38 |             on_ok: Closure::wrap(Box::new(|access: /* Type */| {
   |                                                  ++++++++++++


For more information about this error, try `rustc --explain E0282`.

error: could not compile `midir` (lib) due to 1 previous error
warning: build failed, waiting for other jobs to finish...
Error: `cargo build` failed, see the output above for details
Caused by: `cargo build` failed, see the output above for details
error: Recipe `wasm-build` failed on line 24 with exit code 1

@catilac
Copy link
Copy Markdown
Contributor

catilac commented Apr 14, 2026

Hi @enderversing thank you so much for working on this!

It's important that we keep our PRs focused. This is reformatting the entire README.md which is making it difficult for me to review. Please update your PR to only add the just installation instructions so it can be more easily discussed and merged in.

Additionally, it's important to reference the issue (#111) you're working on. I will update your PR accordingly!

Let me know if you would like to work on this in person, I would be happy to be your guide ✨

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.

Installation of just in README.md

2 participants