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

Components (ESPTOOL-165) #402

Closed
wants to merge 7 commits into from
Closed

Conversation

eykamp
Copy link
Contributor

@eykamp eykamp commented Jan 27, 2019

Builds on my previous PR.

Description of change

Permit passing an already formed esp object to main(), and pull out some functionality into separate functions so it can be called independently from other scripts. Extracting this functionality also makes the program easier to understand.

Should result in no change in functionality when used in current use cases.

Permit passing an already formed esp object to main(), and pull out some functionality into separate functions so it can be called independently from other scripts.  Extracting this functionality also makes the program easier to understand.
@projectgus
Copy link
Contributor

projectgus commented Apr 18, 2019

Refactoring in here LGTM, anything which takes a big block of code and moves it into a function is welcomed!

My only comment is the same one from #400 , to try and keep it possible for esptool.main() to be called as-is.

@radimkarnis radimkarnis changed the title Components Components (ESPTOOL-165) Dec 11, 2020
@radimkarnis
Copy link
Collaborator

Hello @eykamp,
I'm guessing you may have lost some interest over the past 20 months. Do you still want to go on with this PR, or is it okay if I close it? Thank you

Radim

@eykamp
Copy link
Contributor Author

eykamp commented Dec 22, 2020

Well... this project is on hiatus for me, so I have lost a degree of interest. However, it looks like resolving this conflict will be pretty easy, so I'm going to go for it. You can close this if you haven't heard from me in a week or so.

@eykamp
Copy link
Contributor Author

eykamp commented Dec 22, 2020

Regarding the comment from @projectgus, I don't know if it's still relevant. If it is, I don't understand it, but it looks like the requested changes are minor so am happy to resolve it however is good.

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