Skip to content

Conversation

@nathartman
Copy link
Contributor

suggesting removing this sentence bc it isn't technically true and not needed IMO

Please include a summary of the change and what is does. Please also include relevant motivation and context.

Review process:

  • No need to request review from any specific docs team members; we will see your PR and one of us will review.
  • If you need technical review from engineering, please request review from the relevant person.
  • Docs team might commit styling nit fixes to save you the trouble :)
  • You can merge after approval from one docs team member (as well as any necessary technical review).

@nathartman nathartman requested a review from npentrel November 18, 2025 21:34
@viambot viambot added the safe to build This pull request is marked safe to build from a trusted zone label Nov 18, 2025
@netlify
Copy link

netlify bot commented Nov 18, 2025

Deploy Preview for viam-docs ready!

Name Link
🔨 Latest commit d02e64b
🔍 Latest deploy log https://app.netlify.com/projects/viam-docs/deploys/69271b08e15efa000813c118
😎 Deploy Preview https://deploy-preview-4762--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: 61 (🟢 up 23 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 project configuration.

@npentrel npentrel changed the title remove sentence from control-logic.md Remove sentence from control-logic.md Nov 26, 2025
suggesting removing this sentence bc it isn't technically true and not needed IMO
@npentrel npentrel merged commit 24d4ad9 into main Nov 26, 2025
13 checks passed
@npentrel npentrel deleted the nathartman-patch-1 branch November 26, 2025 15:25
@github-actions
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.

4 participants