Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Assets-builder refactoring #376

Merged
merged 12 commits into from
Jun 9, 2024
Merged

Assets-builder refactoring #376

merged 12 commits into from
Jun 9, 2024

Conversation

boozook
Copy link
Owner

@boozook boozook commented Jun 8, 2024

Continuous refactoring, previous part: #366.

  • using new meta-tree instead of build-context by underlying cargo
  • refactoring of assets planning and building

Currently old way is not replaced with new. Both are works, results comparing, used result of the old method.

Rewritten assets-builder benefits:

@github-actions github-actions bot added docs Improvements or additions to documentation cargo About cargo or cargo-playdate labels Jun 8, 2024
@boozook boozook added assets Assets Build System build-utils labels Jun 8, 2024
cargo/src/assets/mod.rs Outdated Show resolved Hide resolved
cargo/src/assets/mod.rs Outdated Show resolved Hide resolved
cargo/src/assets/mod.rs Outdated Show resolved Hide resolved
cargo/src/assets/mod.rs Outdated Show resolved Hide resolved
cargo/src/assets/plan.rs Outdated Show resolved Hide resolved
cargo/src/assets/plan.rs Outdated Show resolved Hide resolved
cargo/src/assets/plan.rs Outdated Show resolved Hide resolved
cargo/src/assets/plan.rs Outdated Show resolved Hide resolved
cargo/src/assets/plan.rs Outdated Show resolved Hide resolved
cargo/src/utils/cargo/meta_deps.rs Outdated Show resolved Hide resolved
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
cargo/src/assets/mod.rs Outdated Show resolved Hide resolved
cargo/src/assets/mod.rs Outdated Show resolved Hide resolved
cargo/src/assets/plan.rs Outdated Show resolved Hide resolved
cargo/src/assets/plan.rs Outdated Show resolved Hide resolved
@boozook boozook marked this pull request as ready for review June 9, 2024 21:27
@github-actions github-actions bot enabled auto-merge (squash) June 9, 2024 21:27
@github-actions github-actions bot merged commit 5a6be83 into main Jun 9, 2024
40 checks passed
@boozook boozook deleted the dev/assets-build branch June 9, 2024 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assets Assets Build System build-utils cargo About cargo or cargo-playdate docs Improvements or additions to documentation
Projects
Status: Done
1 participant