Skip to content
This repository has been archived by the owner on Apr 18, 2022. It is now read-only.

Improvement/reexport ui image load prefab #2181

Merged
merged 6 commits into from Mar 21, 2020

Conversation

azriel91
Copy link
Member

@azriel91 azriel91 commented Mar 21, 2020

#2169 with amended docs and clippy fixes.

PR Checklist

By placing an x in the boxes I certify that I have:

  • Added a changelog entry if this will impact users, or modified more than 5 lines of Rust that wasn't a doc comment.
  • Acknowledged that by making this pull request I release this code under an MIT/Apache 2.0 dual licensing scheme.

If this modified or created any rs files:

  • Ran cargo +stable fmt --all
  • Ran cargo +stable clippy --all --features "vulkan"
  • Ran cargo test --all --features "vulkan"

@codecov
Copy link

codecov bot commented Mar 21, 2020

Codecov Report

Merging #2181 into master will decrease coverage by 0.58%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2181      +/-   ##
==========================================
- Coverage    83.9%   83.32%   -0.59%     
==========================================
  Files          74       74              
  Lines        5996     6074      +78     
==========================================
+ Hits         5031     5061      +30     
- Misses        965     1013      +48
Impacted Files Coverage Δ
amethyst_test/src/in_memory_source.rs 66.66% <0%> (-6.07%) ⬇️
amethyst_core/src/timing.rs 92.66% <0%> (-5.92%) ⬇️
amethyst_assets/src/storage.rs 40.95% <0%> (-2.93%) ⬇️
amethyst_audio/src/sink.rs 97.5% <0%> (-2.5%) ⬇️
amethyst_rendy/src/sprite/mod.rs 93.07% <0%> (-1.83%) ⬇️
amethyst_derive/tests/system_desc.rs 98.24% <0%> (-1.76%) ⬇️
amethyst_audio/src/components/audio_emitter.rs 94.54% <0%> (-1.76%) ⬇️
amethyst_audio/src/output.rs 98.5% <0%> (-1.5%) ⬇️
amethyst_assets/src/dyn_format.rs 83.09% <0%> (-1.19%) ⬇️
amethyst_test/src/wait_for_load.rs 86.51% <0%> (-0.99%) ⬇️
... and 11 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d005aa0...9ee4a91. Read the comment docs.

@azriel91
Copy link
Member Author

bors r+

@bors
Copy link
Contributor

bors bot commented Mar 21, 2020

Build succeeded

@bors bors bot merged commit f8664de into master Mar 21, 2020
@azriel91 azriel91 deleted the improvement/reexport-ui-image-load-prefab branch March 21, 2020 06:14
bors bot added a commit that referenced this pull request Jul 7, 2020
2349: Update book example pong-tutorial-06 to fix clippy warning r=CleanCut a=MrTanoshii

## Description

Clippy will warn of https://rust-lang.github.io/rust-clippy/master/index.html#option_as_ref_deref
The examples were already fixed by #2181 but the book was not.

## Additions

N/A

## Removals

N/A

## Modifications

N/A

## PR Checklist

By placing an x in the boxes I certify that I have:

- [x] Updated the content of the book if this PR would make the book outdated.
- [ ] Added a changelog entry if this will impact users, or modified more than 5 lines of Rust that wasn't a doc comment.
- [ ] Added unit tests for new code added in this PR.
- [x] Acknowledged that by making this pull request I release this code under an MIT/Apache 2.0 dual licensing scheme.

If this modified or created any rs files:

- [x] Ran `cargo +stable fmt --all`
- [x] Ran `cargo clippy --all --features "empty"`
- [x] Ran `cargo test --all --features "empty"`


Co-authored-by: MrTanoshii <47116127+MrTanoshii@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants