Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-03-11 Thread Tomek CEDRO
Alan, I think that some honest discussion may be necessary regarding rules propositions that are blocked only with your -1 vote and quite against the discussion on breaking changes, thus allowing breaking code to be freely merged with the upstream, with no breaking changes handling process, and by

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-03-11 Thread Alan C. Assis
Hi Tomek, My vote -1 was because the breaking text was saying that the author of commit needs to test it and many (all?) arch and boards supported by NuttX. This transfers the responsibility of NuttX QA from the project organization to the end developer, that is unfair. We as project responsible

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-03-07 Thread Alan C. Assis
Alin, but this is not an enforced rule of Linux, see: https://github.com/torvalds/linux/commit/35fa2d88ca9481e5caf533d58b99ca259c63b2fe https://github.com/torvalds/linux/commit/6a774228e890ee04a0ee13f4e6e731ec8554b9c2 https://github.com/torvalds/linux/commit/d0caf9876a1c9f844307effb598ad1312d9e002

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-03-07 Thread Alin Jerpelea
Hi all, I was always looking at the kernel.org is a good development practice since it has a huge community and it has been around for some time https://web.git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/log/Documentation/bpf?h=v6.14-rc5 if we decide that it is too hard to split commi

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-03-07 Thread raiden00pl
I also don't understand this practice of separating documentation from code. When changes in the code have to be reflected in the documentation, what's the point of splitting it into separate commits? If, for example, I do a `git revert`, it may happen that the documentation and code are out of syn

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-03-07 Thread Alin Jerpelea
Hi Alan, development best practices ask for commits to be split in areas they touch Related to that PR: Documentation in not software and does not belong to code changes in arch I think that this rule should be enforced even if we don't do LTS releases for now but we will do them in the future.

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-03-06 Thread Tomek CEDRO
Okay here goes the PR with Guidelines update based on our votes, in draft mode for now, I tired to make it as human friendly as possible, unclear points are marked "TODO", discussion is now open :-) https://github.com/apache/nuttx/pull/15950 Thanks :-) Tomek On Thu, Mar 6, 2025 at 7:39 PM Tome

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-03-06 Thread Tomek CEDRO
Thank you Nathan, yes I will prepare Contribution Guidelines as promised, I am extremely overloaded recently sorry for the delay!! Tomek On Sun, Mar 2, 2025 at 5:04 PM Nathan Hartman wrote: > > Hi all, > > A very *BIG* THANK YOU! to Tomek for driving this and doing the work > of running the votes

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-03-02 Thread Nathan Hartman
Hi all, A very *BIG* THANK YOU! to Tomek for driving this and doing the work of running the votes and tallying the results! Also THANK YOU to everyone who has participated so far in the discussions and voting. I guess the next steps will be to make any updates to the Contributing Guidelines? Th

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-03-01 Thread Alan C. Assis
Thank Alin, but in the ROUND 2 was written in *separated PR* Yes, I think all commits should be separated into logic parts, depending on which function is implemented. For example, a new generic driver shouldn't include arch/ and boards/ files in the same commit, but a fix that involved arch/, bo

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-03-01 Thread Alin Jerpelea
Hi Alan, just for clarification Documentation should be part of the same PE but in a separate commit Best regards Alin On Sat, 1 Mar 2025, 11:11 Alan C. Assis, wrote: > Hi Tomek, > > Nice work! Kudos! > > About Documentation I voted +1 to have Documentation, but later I realized > it was requ

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-03-01 Thread Alan C. Assis
Hi Tomek, Nice work! Kudos! About Documentation I voted +1 to have Documentation, but later I realized it was required to be in a separate PR, which I don't agree with. When I contributor submit a new feature that doesn't have Documentation I ask them to include it to avoid creating hidden featu

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-03-01 Thread Tomek CEDRO
Allright Ladies and Gentlemen, here goes the results :-) ACCEPTED: 1. Contributing Guidelines with hints for Reviewers. 2. PR and GIT COMMITS must adhere to Contributing Guidelines or rejected. 3. Git commit messages as important as PR description. 4. Proper description details requirements. 5. PR

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-03-01 Thread Alin Jerpelea
Hi Tomek, thanks for conducting the vote and expressing your feelings. As always you point the obvious things and I hope that we will aim to persuit standardisation and compatibility. Have a nice weekend Alin On Sat, 1 Mar 2025, 08:00 Tomek CEDRO, wrote: > My thoughts and comments that I did

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-28 Thread Tomek CEDRO
My thoughts and comments that I did not want to put as part of the results message: It seems like this voting revealed two mindsets - one wants quick and dirty experimental changes with low bar for acceptance that may be streamed up from a single big organization that is probably paid for the amou

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-28 Thread Matteo Golin
On Fri, Feb 28, 2025 at 1:45 AM Dmitri Shilov wrote: > > > > > > 1. Contributing Guidelines with hints for Reviewers. > > > > > > > We are adding additional section for Reviewers to Contributing > > > > Guidelines in order to provide checklist and complementary set of > > > > rules that s

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-28 Thread Tim Hardisty
S unclear how to vote. Reply to any reply? With different formatting and (mis)quotes? So i will not quote and you can sort it out I hope!! TimH: +1 to all - on the basis that if this doesn't work out quite right it will be reviewed and changed. Best to try something like this than not - it

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-28 Thread Tomek CEDRO
Ladies and Gentlemen, 3h to voting close :-P -- CeDeROM, SQ7MHZ, http://www.tomek.cedro.info

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-27 Thread Dmitri Shilov
> > > > 1. Contributing Guidelines with hints for Reviewers. > > > > > We are adding additional section for Reviewers to Contributing > > > Guidelines in order to provide checklist and complementary set of > > > rules that should filter out breaking code as much as possible also > > > on o

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-27 Thread Alin Jerpelea
Hi Tomek, please ask people that voted on google forms to recast the vote. We shouod not not bring votes from extern sources because those are not archived Best regards Alin On Thu, 27 Feb 2025, 18:52 Tomek CEDRO, wrote: > Thanks Alan, I will gather votes from google forms too and present > ev

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-27 Thread Alan C. Assis
Thank Tomek, yes I think using the mailing list is better, to avoid a single point of failure. BR, Alan On Thu, Feb 27, 2025 at 2:51 PM Tomek CEDRO wrote: > Thanks Alan, I will gather votes from google forms too and present > everything on the mailing list :-) > We will use only mailing list f

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-27 Thread Tomek CEDRO
Thanks Alan, I will gather votes from google forms too and present everything on the mailing list :-) We will use only mailing list for voting as Alin noted :-) Tomek On Thu, Feb 27, 2025 at 3:46 PM Alan C. Assis wrote: > > > > Google forms confirmed that I voted on Fev 24, no idea why you didn

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-27 Thread Nathan Hartman
On Thu, Feb 27, 2025 at 8:30 AM Sebastien Lorquet wrote: > > > On 27/02/2025 13:53, Filipe Cavalcanti wrote: > > 1. Contributing Guidelines with hints for Reviewers. > > > We are adding additional section for Reviewers to Contributing > > Guidelines in order to provide checklist and complementar

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-27 Thread Alan C. Assis
If you find the first answered form, please use it. BR, Alan On Thu, Feb 27, 2025 at 11:46 AM Alan C. Assis wrote: > [image: image.png] > > Google forms confirmed that I voted on Fev 24, no idea why you didn't see > it... > > > On Thu, Feb 27, 2025 at 11:44 AM Alan C. Assis wrote: > >> Just d

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-27 Thread Alan C. Assis
[image: image.png] Google forms confirmed that I voted on Fev 24, no idea why you didn't see it... On Thu, Feb 27, 2025 at 11:44 AM Alan C. Assis wrote: > Just did it again! > > On Thu, Feb 27, 2025 at 11:33 AM Alan C. Assis wrote: > >> Hi Tomek, it is strange, I voted in this second round, b

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-27 Thread Alan C. Assis
Just did it again! On Thu, Feb 27, 2025 at 11:33 AM Alan C. Assis wrote: > Hi Tomek, it is strange, I voted in this second round, but my vote didn't > show up. > > BR, > > Alan > > On Thu, Feb 27, 2025 at 8:20 AM Tomek CEDRO wrote: > >> Dear NuttX Community please vote! Tomorrow we are closing

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-27 Thread Alan C. Assis
Hi Tomek, it is strange, I voted in this second round, but my vote didn't show up. BR, Alan On Thu, Feb 27, 2025 at 8:20 AM Tomek CEDRO wrote: > Dear NuttX Community please vote! Tomorrow we are closing the vote and > we have only 4 votes for now, in the previous round we had 12 votes, > so vo

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-27 Thread Sebastien Lorquet
hink this is priority right now? We can tweak the guideline later. 0 Filipe -1 Sebastien NERVER EVER let untrusted and unverified code in without a review even if the process is slowed down. If developer WANTS his code merged the burden is on them to get other reviewers involved so merge bec

Re: Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-27 Thread Filipe Cavalcanti
s (if applicable). > 3. After two weeks ask for the independent reviewer to merge if there > are no other reviews. The independent reviewer is responsible for > checking if the PR matches the *Lazy Consensus* conditions before > merging it. -1: Tomek (Considering we are leaving 2 rev

Re: Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-27 Thread Lee, Lup Yuen
1. Contributing Guidelines with hints for Reviewers. > We are adding additional section for Reviewers to Contributing > Guidelines in order to provide checklist and complementary set of > rules that should filter out breaking code as much as possible also on > our side. +1 Tomek +1 Alin +1 Tiago +

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-27 Thread Tomek CEDRO
Dear NuttX Community please vote! Tomorrow we are closing the vote and we have only 4 votes for now, in the previous round we had 12 votes, so vote vote :-) -- CeDeROM, SQ7MHZ, http://www.tomek.cedro.info

RE: Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-27 Thread Tim Kan
Hi, Voting again (my previous e-mail was discarded in the voting process): 1. Contributing Guidelines with hints for Reviewers. > We are adding additional section for Reviewers to Contributing > Guidelines in order to provide checklist and complementary set of > rules that should filter out break

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-25 Thread Tiago Medicci Serrano
Hi, Voting again (my previous e-mail was discarded in the voting process): 1. Contributing Guidelines with hints for Reviewers. > We are adding additional section for Reviewers to Contributing > Guidelines in order to provide checklist and complementary set of > rules that should filter out break

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-24 Thread Alin Jerpelea
I am sorry for the confusion Please fill in your answers to this email followed by the voter's name to avoid future confusion 1. Contributing Guidelines with hints for Reviewers. We are adding additional section for Reviewers to Contributing Guidelines in order to provide checklist and complement

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-24 Thread Nathan Hartman
I think we should each reply to the original vote email. If we reply to someone else's vote, the votes will get all mixed up like that. On Mon, Feb 24, 2025 at 9:26 AM Tiago Medicci Serrano wrote: > > Hi Alin, > > It seems you mixed my answers and Tomek's together. Can you double-check, > please?

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-24 Thread Tiago Medicci Serrano
Hi Alin, It seems you mixed my answers and Tomek's together. Can you double-check, please? Some answers weren't answered by you (8, for instance). Best regards, Em seg., 24 de fev. de 2025 às 11:12, Alin Jerpelea escreveu: > On Mon, Feb 24, 2025 at 2:26 PM Tomek CEDRO wrote: > > > My respons

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-24 Thread Alin Jerpelea
On Mon, Feb 24, 2025 at 2:26 PM Tomek CEDRO wrote: > My responses below :-) > > On Mon, Feb 24, 2025 at 1:57 PM Tomek CEDRO wrote: > > 1. Contributing Guidelines with hints for Reviewers. > > We are adding additional section for Reviewers to Contributing > > Guidelines in order to provide checkl

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-24 Thread Tiago Medicci Serrano
Hi, > 1. Contributing Guidelines with hints for Reviewers. > We are adding additional section for Reviewers to Contributing > Guidelines in order to provide checklist and complementary set of > rules that should filter out breaking code as much as possible also on > our side. +1 > 2. PR and GIT

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-24 Thread Tomek CEDRO
My responses below :-) On Mon, Feb 24, 2025 at 1:57 PM Tomek CEDRO wrote: > 1. Contributing Guidelines with hints for Reviewers. > We are adding additional section for Reviewers to Contributing > Guidelines in order to provide checklist and complementary set of > rules that should filter out brea

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-24 Thread Tomek CEDRO
On Sat, Feb 22, 2025 at 3:45 AM Tomek CEDRO wrote: > Hello world :-) > After the first voting, discussions, and updates, here goes the Round > 2 of the voting for NuttX Contributing Guidelines update: > https://forms.gle/m3uRDGuE3QZy2yNn6 > There are again all rule propositions numbered from 1 to

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-24 Thread alin.jerpe...@sony.com
round 1) Best regards Alin Från: Tiago Medicci Serrano Skickat: den 24 februari 2025 12:51 Till: dev@nuttx.apache.org Ämne: Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502. Voted! Please vote. This is valid for the overall NuttX community (not only

Re: [VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-24 Thread Tiago Medicci Serrano
Voted! Please vote. This is valid for the overall NuttX community (not only PMCs or committers). Em sex., 21 de fev. de 2025 às 23:45, Tomek CEDRO escreveu: > Hello world :-) > > After the first voting, discussions, and updates, here goes the Round > 2 of the voting for NuttX Contributing Guide

[VOTE] ROUND2 NuttX Contributing Guidelines update 202502.

2025-02-21 Thread Tomek CEDRO
Hello world :-) After the first voting, discussions, and updates, here goes the Round 2 of the voting for NuttX Contributing Guidelines update: https://forms.gle/m3uRDGuE3QZy2yNn6 There are again all rule propositions numbered from 1 to 19 with optional text fields if you feel texting can be fix