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

Added node.info() #3655

Open
wants to merge 3 commits into
base: dev-esp32
Choose a base branch
from
Open

Conversation

serg3295
Copy link

@serg3295 serg3295 commented Jun 21, 2024

Fixes #2759

  • This PR is for the dev branch rather than for the release branch.
  • This PR is compliant with the other contributing guidelines as well (if not, please describe why).
  • I have thoroughly tested my contribution.
  • The code changes are reflected in the documentation at docs/*.

This PR adds support for the previously missing node.info() method on esp32.

Test code:

function printTable(t) for k, v in pairs(t) do print(k, v) end end

print("\n====== no params:") printTable(node.info())
print("\n====== build config:") printTable(node.info("build_config"))
print("\n====== lfs:") printTable(node.info("lfs"))
print("\n====== hw:") printTable(node.info("hw"))
print("\n====== sw version:") printTable(node.info("sw_version"))

@pjsg
Copy link
Member

pjsg commented Jun 21, 2024

LGTM

and format code to make it readable.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants