Skip to content

Keep Build Files In build#89

Merged
tavianator merged 3 commits intotavianator:mainfrom
ElectronicsArchiver:Build
Apr 16, 2022
Merged

Keep Build Files In build#89
tavianator merged 3 commits intotavianator:mainfrom
ElectronicsArchiver:Build

Conversation

@ElectronicsArchiver
Copy link
Copy Markdown
Contributor

  • Automatically creates a /build/ folder
  • .o & .d files are generated in /build/

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 16, 2022

Codecov Report

Merging #89 (ef1d3bd) into main (33cc3b9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #89   +/-   ##
=======================================
  Coverage   63.43%   63.43%           
=======================================
  Files           3        3           
  Lines         134      134           
  Branches        7        7           
=======================================
  Hits           85       85           
  Misses         44       44           
  Partials        5        5           

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 33cc3b9...ef1d3bd. Read the comment docs.

Copy link
Copy Markdown
Owner

@tavianator tavianator left a comment

Choose a reason for hiding this comment

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

Thanks for doing this! Just a couple nits.

Comment thread Makefile Outdated
Comment thread Makefile Outdated
@ElectronicsArchiver
Copy link
Copy Markdown
Contributor Author

Not sure, does it work with 2 paths or do you need to separate it~

@tavianator
Copy link
Copy Markdown
Owner

You mean $(wildcard)? It works with two patterns, I just tested it.

@tavianator
Copy link
Copy Markdown
Owner

make clean needs to be updated. After that I think it's good to go!

@tavianator tavianator merged commit 521ba65 into tavianator:main Apr 16, 2022
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.

2 participants