Skip to content

DOCS-3797: Modernize the Micro-RDK platform #4212

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

Merged
merged 3 commits into from
Apr 16, 2025

Conversation

JessamyT
Copy link
Collaborator

@JessamyT JessamyT commented Apr 11, 2025

Swept for instances of related versions and just found these two. We of course point to https://github.com/viamrobotics/micro-rdk/tree/main/templates/project but that is being updated.

Don't merge until https://viam.atlassian.net/browse/RSDK-8990 is released.

@JessamyT JessamyT requested a review from acmorrow April 11, 2025 00:44
@viambot viambot added the safe to build This pull request is marked safe to build from a trusted zone label Apr 11, 2025
Copy link

netlify bot commented Apr 11, 2025

Deploy Preview for viam-docs ready!

Name Link
🔨 Latest commit 13779d5
🔍 Latest deploy log https://app.netlify.com/sites/viam-docs/deploys/67ffefc17076260008a3021c
😎 Deploy Preview https://deploy-preview-4212--viam-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 28 (🔴 down 16 from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 92 (no change from production)
PWA: 70 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@nathan-contino nathan-contino left a comment

Choose a reason for hiding this comment

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

LGTM, small sensible changes!

@acmorrow
Copy link
Member

Please hold off on merging this: we are in a code freeze right now as we try to land the ESP-IDF 5 upgrade. I'll approve when we are unfrozen.

Copy link
Member

@acmorrow acmorrow left a comment

Choose a reason for hiding this comment

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

I think we should merge this now that main (which is the branch we indicate in these docs for using for project/module generation) is based on ESP-IDF 5.

One small change in the link to espressif though, which should reference ESP-IDF v5.4, rather than latest.

@JessamyT JessamyT merged commit 9f0220c into viamrobotics:main Apr 16, 2025
12 checks passed
@JessamyT JessamyT deleted the 3797micromodern branch April 16, 2025 18:09
Copy link

🔎💬 Inkeep AI search and chat service is syncing content for source 'Viam Docs'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe to build This pull request is marked safe to build from a trusted zone
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants