Hi Igniters, > I think this guide should be much shorter and simple.
As you know we also have CONTRIBUTING.md [1] on GitHub. Perhaps it can be the shortest one for newcomers. [1] https://github.com/apache/ignite/blob/master/CONTRIBUTING.md 2021-03-15 21:56 GMT+03:00, Kseniya Romanova <romanova.ks....@gmail.com>: >> >> > Components and their maintainers >> > It seems that this list should be updated too. >> I would be glad if somebody does it, but I don't have any more >> information >> to fill there. > > > I'll be happy to collect information and update this section. But I also > think this should be a separate page (and howto should have a link on it). > > пн, 15 мар. 2021 г. в 18:25, Pavel Tupitsyn <ptupit...@apache.org>: > >> Ilya, >> >> Thanks for the effort! >> >> I think this guide should be much shorter and simple. >> Right now it is intimidating for newcomers. >> >> What they need is basically >> * Register in Jira, pick a ticket, assign, put In Progress >> * Create a fork, implement >> * Create a PR >> * Ask for review >> >> Maybe we should have a separate, detailed guide for Committers, >> and a simple one for Contributors? >> >> On Mon, Mar 15, 2021 at 6:19 PM Ilya Kasnacheev >> <ilya.kasnach...@gmail.com >> > >> wrote: >> >> > Hello! >> > >> > Please see inline. >> > >> > пн, 15 мар. 2021 г. в 18:06, Maxim Muzafarov <mmu...@apache.org>: >> > >> > > Hello, >> > > >> > > >> > > > Ignite employs both Review-Then-Commit processes. >> > > >> > > The Commit-Then-Review (CTR) removed? >> > > >> > I don't see any applications of CTR during the few last years. >> > Streamers >> > were supposed to be CTR but Saikat Maitra still asked for the review of >> > streamers-related commits. >> > >> > > Information for committers >> > > >> > > Do we need this on a page for newcomers? I'd like to mention that >> > > some >> > > of the committers still use the commit script, however, I think it >> > > will be better to configure the GitHub interaction. >> > > >> > I don't think there's a separate page for committers. If there is, >> > please >> > point me to it, and we can remove the section. I don't think we should >> > be >> > writing two pages at once, so I decided not to drop any essential >> > information. >> > >> > > Components and their maintainers >> > > >> > > It seems that this list should be updated too. >> > > >> > I would be glad if somebody does it, but I don't have any more >> information >> > to fill there. >> > >> > >> > > > Working on a ticket >> > > I think we should mention the Intellij IDEA checkstyle plugin and its >> > > configuration (importation of checkstyle.xml to the IDE). >> > > >> > I would be glad if somebody contributes to it, or we may just provide a >> > link to coding guidelines and mention it there. >> > >> > >> > >> > > > GIT workflow >> > > >> > > Do we need it? >> > > >> > I think we do, this workflow is non-trivial and I don't think it is >> > documented anywhere. We can get rid of ASCII art section, though. >> > >> > WDYT? >> > >> > Regards, >> > >> > >> > > >> > > >> > > On Mon, 15 Mar 2021 at 17:25, Ilya Kasnacheev < >> ilya.kasnach...@gmail.com >> > > >> > > wrote: >> > > > >> > > > Hello! >> > > > >> > > > When adding new users to the Contributor role, we usually give them >> > > > a >> > > link >> > > > to "How to Contribute" wiki page. >> > > > >> > > > However, I was feeling that it was in many ways outdated, referring >> to >> > > > outdated development practices and not emphasising TC tests and >> > > > MTCGA >> > > bot. >> > > > >> > > > So we took liberty to rewrite this page, meet >> > > > >> > > >> > >> https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute+2021 >> > > > >> > > > We tried to streamline it, make it more friendly to newcomers and >> just >> > > > shorter. >> > > > >> > > > Please check it out, share your feelings. >> > > > >> > > > I plan to replace the legacy >> > > > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute >> > > with >> > > > this page based on your feedback.. >> > > > >> > > > Regards, >> > > > -- >> > > > Ilya Kasnacheev >> > > >> > >> > -- Best regards, Ivan Pavlukhin
