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

Fix 0.12 template #101

Closed
wants to merge 3 commits into from
Closed

Fix 0.12 template #101

wants to merge 3 commits into from

Conversation

0x6273
Copy link
Contributor

@0x6273 0x6273 commented Dec 12, 2019

Fixes warnings about unused imports in the 0.12 template, and changes the asset path to "assets" instead of the root of the project dir. This was originally part of #98 but was moved to a separate pr as requested.

trying to build a project created with `amethyst new -a 0.12.0` would
raise warnings about usused imports.
@0x6273 0x6273 mentioned this pull request Dec 12, 2019
@codecov-io
Copy link

codecov-io commented Dec 12, 2019

Codecov Report

Merging #101 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #101   +/-   ##
=======================================
  Coverage   94.11%   94.11%           
=======================================
  Files           3        3           
  Lines          34       34           
=======================================
  Hits           32       32           
  Misses          2        2

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 7d7c147...149ed31. Read the comment docs.

bors bot added a commit that referenced this pull request Dec 26, 2019
98: Add 0.13 template r=jojolepro a=0x6273

~~Fixes warnings about unused imports in the 0.12 template, and adds a 0.13.2 template (identical to the 0.12 one).~~

Adds a 0.13 template. Some things were moved into a separate pr #101 as requested by a reviewer.

Fixes #97 

Co-authored-by: 0x40 <0x40@keemail.me>
Copy link
Contributor

@AnneKitsune AnneKitsune left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors r+

Thanks!

bors bot added a commit that referenced this pull request Dec 26, 2019
101: Fix 0.12 template r=jojolepro a=0x6273

Fixes warnings about unused imports in the 0.12 template, and changes the asset path to "assets" instead of the root of the project dir. This was originally part of #98 but was moved to a separate pr as requested.

Co-authored-by: 0x40 <0x40@keemail.me>
@bors
Copy link
Contributor

bors bot commented Dec 26, 2019

Build failed

bors bot added a commit that referenced this pull request Mar 24, 2020
98: Add 0.13 template r=azriel91 a=0x6273

~~Fixes warnings about unused imports in the 0.12 template, and adds a 0.13.2 template (identical to the 0.12 one).~~

Adds a 0.13 template. Some things were moved into a separate pr #101 as requested by a reviewer.

Fixes #97 

Co-authored-by: 0x40 <0x40@keemail.me>
@CleanCut
Copy link
Member

👋 I'm closing all PRs and Issues since amethyst_tools is being end-of-life'd. Please see this forum post for a place to respond and/or discuss the situation rather than here on the repo.

@CleanCut CleanCut closed this Aug 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants