Skip to content

Add CI to block the commit of large files #24147

@alice-i-cecile

Description

@alice-i-cecile

What problem does this solve or what need does it fill?

Large files permanently bloat repo size. This sucks for contributors, and is extremely painful to fix (see #24130).

@mockersf has been a strong advocate of this, and @andriyDev and @atlv24 have both suggested blocking this in CI.

What solution would you like?

Add a CI check that scans for large new files (over 1 MB or something?) and rejects the PR if any are detected.

The error should include clear advice on what should be done instead. Currently we're using https://github.com/bevyengine/bevy_asset_files, which is moderately cursed but does work.

This is probably fine as an on-merge check only.

What alternative(s) have you considered?

Force push to main 😎

Metadata

Metadata

Assignees

No one assigned

    Labels

    A-Build-SystemRelated to build systems or continuous integrationC-FeatureA new feature, making something new possibleS-Ready-For-ImplementationThis issue is ready for an implementation PR. Go for it!

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions