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

Tools: Extract resources object and implement incremental scan (original: #7183) #19

Open
wants to merge 44 commits into
base: base_of_7183
Choose a base branch
from

Conversation

makimat
Copy link

@makimat makimat commented Oct 30, 2018

Description

I found that the resource object being part of the tool chain, and depending on
tool chain infrastructure was a bit odd, so I extracted resources from tool
chains. The resulting code is quite a bit more clear in it's purpose and can be a
tad smaller (extra documentation and formatting excluded). I made the resources
object fully incremental, meaning that the scan stops early otfen for targets,
toolchains and features, and can easily pick itself back up where it left off.

...

Pull request type

[ ] Fix
[x] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

@softagram-bot
Copy link

Softagram Impact Report for pull/19 (head commit: 750a2ac)

Target: K64F

⭐ Visual Overview

Changed elements and changed dependencies.
Changed dependencies - click for full size
Graph legend
(Open in Softagram Desktop for full details)

⭐ Change Impact

How the changed files are used by the rest of the project
Impacted files - click for full size
Graph legend
(Open in Softagram Desktop for full details)

📄 Full report

Give feedback of this report to support@softagram.com

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.

3 participants