Skip to content

added author, part number explained article #422

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

added author, part number explained article #422

wants to merge 1 commit into from

Conversation

@FBEZ
Copy link
Collaborator

@FBEZ FBEZ commented Mar 6, 2025 •
edited
Loading

Reviewed Internally

Description

Related

Testing

Checklist

Before submitting a Pull Request, please ensure the following:

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

@f-hollow
Copy link
Collaborator

f-hollow commented Mar 7, 2025

As our pr-check-links job experiences a bug, I checked the links locally. This article is clean.

@FBEZ FBEZ added the GitLab-Sync-Merge Contribution reviewed and synced from the internal GitLab mirror label Mar 7, 2025
@pedrominatel pedrominatel self-requested a review March 7, 2025 10:17
pedrominatel
pedrominatel approved these changes Mar 7, 2025
View reviewed changes
Copy link
Member

@pedrominatel pedrominatel left a comment

Choose a reason for hiding this comment

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

LGTM.

@github-actions
Copy link

github-actions bot commented Mar 17, 2025

🎉 A preview for this PR is available at: https://preview-developer.espressif.com/pr422/

@FBEZ FBEZ force-pushed the espressif-part-number-explained-int branch from 832dcaf to e8d2905 Compare March 17, 2025 07:11
@pedrominatel
Copy link
Member

pedrominatel commented Mar 17, 2025

I'll close this PR due to the new one #441.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pedrominatel pedrominatel pedrominatel approved these changes

@f-hollow f-hollow f-hollow approved these changes

Assignees

No one assigned

Labels

GitLab-Sync-Merge Contribution reviewed and synced from the internal GitLab mirror

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@FBEZ @f-hollow @pedrominatel