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
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Validate that all assets can be loaded in CI #746

wants to merge 10 commits into from

Conversation

@alice-i-cecile
Copy link
Contributor

@alice-i-cecile alice-i-cecile commented Apr 20, 2023 •
edited
Loading

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 alice-i-cecile mentioned this pull request May 6, 2023
Open
@alice-i-cecile
Copy link
Contributor Author

alice-i-cecile commented May 7, 2023

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
Contributor Author

alice-i-cecile commented May 7, 2023

Probably blocked on bevyengine/bevy#8566

@alice-i-cecile alice-i-cecile mentioned this pull request May 8, 2023
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

automation-and-testing Automate everything

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@alice-i-cecile