Also impose a size limit for Wasm files

Open Beginner friendly
  • Is something wrong?
#94 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
66/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Active
Tech stack
rust, wasm
Domain
tooling

Research direction

Start by reading src/check/files.rs at lines 132–136, where file checks are handled. Determine how the existing checks apply to Wasm files and define a suitable size limit; done means oversized Wasm files are flagged while files within the limit pass.

Written by the indexing model from the issue text.

Description

While large Wasm files may be necessary for functionality, we still want to impose a limit and currently this means the human reviewer needs to be very careful.

https://github.com/typst/package-check/blob/3abdb3cddc477fb00f75126bb26dd98dd90514c8/src/check/files.rs#L132-L136

Dominant language
Rust
Stars
37
Forks
9
Avg merge
7d 3h
Merged PRs (30d)
6

Getting set up

This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from typst/package-check

All issues in typst/package-check

Similar issues

More Rust issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.