Skip to content
This repository has been 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

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

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.
1 participant
@alice-i-cecile