Skip to content
This repository was archived by the owner on Nov 22, 2023. It is now read-only.

Validate that all assets can be loaded in CI - #746

Draft
alice-i-cecile wants to merge 10 commits into
mainfrom
assets-all-load
Draft

Validate that all assets can be loaded in CI#746
alice-i-cecile wants to merge 10 commits into
mainfrom
assets-all-load

Conversation

@alice-i-cecile

@alice-i-cecile alice-i-cecile commented Apr 20, 2023

Copy link
Copy Markdown
Contributor

Motivated by #736: this is the first step towards more complex asset validation.

Leaving as a draft for now and slowly chipping away at it. Feel free to adopt it if the urge strikes you.

Status

  • build skeleton for CI
  • find the folder for the assets
  • start loading the assets
  • add this job to the list of CI tasks
  • ensure that the assets all load. Currently failing on PopOS, but working for PaulH and myself on Windows here on Discord
  • ensure that this doesn't require a GPU by extracting out the asset loaders

@alice-i-cecile alice-i-cecile added the automation-and-testing Automate everything label Apr 20, 2023
@alice-i-cecile

Copy link
Copy Markdown
Contributor Author

See docs at https://docs.rs/bevy/latest/bevy/asset/struct.AssetServer.html#method.load for more details on the asset path nonsense.

@alice-i-cecile

Copy link
Copy Markdown
Contributor Author

Probably blocked on bevyengine/bevy#8566

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

automation-and-testing Automate everything

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant