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

coding_standard.md: Minor typos, fixes #1

Merged
merged 1 commit into from
Feb 19, 2025

Conversation

rzr
Copy link
Contributor

@rzr rzr commented Feb 17, 2025

I am making a small change to check if this repo
is open to user contributions.

If merged i will propose a more substantial change.

I am making a small change to check if this repo
is open to user contributions.

If merged i will propose a more substantial change.

Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
@silabs-Tabi silabs-Tabi requested a review from a team February 18, 2025 19:33
@conq
Copy link
Contributor

conq commented Feb 19, 2025

Thank you for fixing the typos! :)

@conq conq merged commit 23f1a6b into SiliconLabsSoftware:main Feb 19, 2025
@rzr
Copy link
Contributor Author

rzr commented Feb 20, 2025

Thank you for fixing the typos! :)

Thanks, It was just a trigger :)

Let me push a couple of other suggestions, is this also the right place recommendations regarding OSS consumption/productions ?

I have lot of content elsewhere (at sl), but this time i´d like it to be deeply reviewed and approved.

Let's start with:

#2

ps: Just by curiosity, may I ask merge was preferred over a ff-merge (rebase) ? is there any benefit ?

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.

2 participants