Hi Jark, Thanks for driving the effort to integrate the Chinese website!
We have the policy that new features / improvements should be documented in the same PR for a long time. So far, this was checked by reviewers and committers but often overlooked or decided to add documentation in a subsequent PR. When we introduced the PR template, we added "Documentation" as a dedicated section to remind contributors (and reviewers) about the documentation policy. I think adding an additional step in the review process to check the documentation would help to enforce the policy. +1 for this proposal. IMO, this is independent of the Chinese documentation, but it would certainly help to keep both versions in sync. Best, Fabian Am Do., 21. März 2019 um 06:10 Uhr schrieb Jark Wu <imj...@gmail.com>: > Hi all, > > In the past discussion of Supporting Chinese Documentation for Apache > Flink[1], we reach a consensus to add a documentation check item to the > flinkbot review process. > > I propose the idea here to get some more feedbacks about this. > > The new item we want to add is: > > ``` > ### 6. Are English and Chinese documentation updated? > > If the pull request introduces a new feature, the feature should be > documented. The Flink community is maintaining both English and Chinese > documents. So both English and Chinese documentation should be updated. If > you are not familiar with Chinese language, please open a JIRA tagged with > the `chinese-translation` component for Chinese documentation translation > and link it with current JIRA issue. If you are familiar with Chinese > language, you are encouraged to update both sides in one pull request. > ``` > > We have opened a pull request [2] to update it to the website. > > What do you think about this? > > Thanks, > Jark > > > [1] > > http://apache-flink-mailing-list-archive.1008284.n3.nabble.com/DISCUSS-Contributing-Chinese-website-and-docs-to-Apache-Flink-tt26603.html#a26890 > [2] https://github.com/apache/flink-web/pull/190 > > On Thu, 7 Mar 2019 at 21:48, Robert Metzger <rmetz...@apache.org> wrote: > > > Each Jira ticket has a "last updated" field, and in a JIRA search, you > can > > sort results by that field. > > > > So I will regularly check all Jira tickets which have been updated since > > the last time my tool checked. For all changed Jira tickets, I'll update > > the PR if the component has changed. > > > > The implementation will be a bit differently, to not run into rate limits > > with the JIRA or GitHub API. > > > > On Thu, Mar 7, 2019 at 2:40 PM Chesnay Schepler <ches...@apache.org> > > wrote: > > > > > How do you intend to keep the label up-to-date with whatever > > > modifications are made in JIRA? > > > > > > On 07.03.2019 13:40, Robert Metzger wrote: > > > > I will automatically assign the Jira component as a label to the PR, > > yes. > > > > You won't have to manually update the label on the PR, this will be > > done > > > > automatically. > > > > > > > > So JIRA will stay the ground truth for setting the component > correctly. > > > > > > > > On Thu, Mar 7, 2019 at 10:11 AM Chesnay Schepler <ches...@apache.org > > > > > wrote: > > > > > > > >> Component labels seem a bit redundant. Every JIRA with an open PR > > > >> already has a "pull-request-available" tag. > > > >> So this information already exists. > > > >> > > > >> I assume you'll base the labels on the component tags at the time > the > > PR > > > >> is opened, but this also implies that they may be set incorrectly > (or > > > >> not at all) by the contributor. In this case we now have to update > the > > > >> component both in JIRA and on GitHub, and I'm most certainly not > > looking > > > >> forward to that. > > > >> > > > >> On 06.03.2019 13:51, Robert Metzger wrote: > > > >>> This is the picture: > > > >>> > > > >> > > > > > > https://user-images.githubusercontent.com/89049/53882383-7fda9380-4016-11e9-877d-10cdc00bdfbd.png > > > >>> Speaking about feature requests, priorities and time-spend: My plan > > was > > > >> to > > > >>> now work on introducing a new label category for the components. > > > >>> This should get us a lot better overview over the per-component > > > >>> status/health of pull requests. > > > >>> > > > >>> > > > >>> On Wed, Mar 6, 2019 at 12:58 PM Chesnay Schepler < > ches...@apache.org > > > > > > >> wrote: > > > >>>> The image didn't go through. > > > >>>> > > > >>>> I would keep it as is; imo there are significantly more important > > > things > > > >>>> that I'd like Robert to spend time on. (literally everything in > the > > > >>>> Feature requests section) > > > >>>> > > > >>>> If we want to better distinguish new PRs I would suggest to either > > a) > > > >>>> introduce a dedicated "New" label or b) not attach any label by > > > default, > > > >>>> and only attach the description label if someone has > > > >>>> approved/disapproved it. > > > >>>> > > > >>>> On 06.03.2019 12:37, Robert Metzger wrote: > > > >>>>> Hey Kurt, > > > >>>>> thanks a lot for this idea. > > > >>>>> > > > >>>>> My reasoning behind using just one color is the following: I > wanted > > > to > > > >>>>> use one color per category of labels. > > > >>>>> So when we are introducing labels for components, that it'll look > > > like > > > >>>>> this: > > > >>>>> > > > >>>>> image.png > > > >>>>> > > > >>>>> But we could of course also go with color families per category. > So > > > >>>>> "review" is green colors, "component" is red colors and so on. > > > >>>>> > > > >>>>> If nobody objects (or agrees) with me, I'll change the colors > soon. > > > >>>>> > > > >>>>> > > > >>>>> On Wed, Mar 6, 2019 at 7:51 AM Kurt Young <ykt...@gmail.com > > > >>>>> <mailto:ykt...@gmail.com>> wrote: > > > >>>>> > > > >>>>> Hi Dev, > > > >>>>> > > > >>>>> I've been using the flinkbot and the label for a couple > days, > > > it > > > >>>>> worked > > > >>>>> really well! I have a minor suggestion, can we > > > >>>>> use different colors for different labels? We don't need to > > > have > > > >>>>> different > > > >>>>> colors for every label, but only to distinguish whether > > > >>>>> someone had review the PR. > > > >>>>> For example, "review=description?" is the initial default > > > label, > > > >>>>> and it may > > > >>>>> indicate that no reviewer has been try to review it. > > > >>>>> > > > >>>>> For "review=architecture?", "review=consensus?", > > > >>>>> "review=quality?", they > > > >>>>> indicate that at least someone has try to review it and > > > >>>>> approved something. It sounds like the review is in > progress. > > > >>>>> > > > >>>>> For "review=approved ✅", it indicates the review is > finished. > > > >>>>> > > > >>>>> So i think 3 colors is enough, it tell committers whether > the > > > >>>>> review has > > > >>>>> not started yes, or in progress, or is finished. > > > >>>>> > > > >>>>> What do you think? > > > >>>>> > > > >>>>> Best, > > > >>>>> Kurt > > > >>>>> > > > >>>>> > > > >>>>> On Mon, Mar 4, 2019 at 6:50 PM Robert Metzger < > > > >> rmetz...@apache.org > > > >>>>> <mailto:rmetz...@apache.org>> wrote: > > > >>>>> > > > >>>>> > GitHub has two methods for authentication with the APIs: > > > >>>>> > a) using an account's oauth token > > > >>>>> > b) using the GitHub Apps API > > > >>>>> > > > > >>>>> > Most of the libraries for the GH API use a), so does > > > Flinkbot. > > > >>>>> The problem > > > >>>>> > with a) is that it does not allow for fine-grained access > > > >>>>> control, and > > > >>>>> > Infra does not want to give Flinkbot "write" access to > > > >>>>> "apache/flink". > > > >>>>> > That's why I need to rewrite parts of the bot to support > > b), > > > >>>>> which allows > > > >>>>> > to give access only a repo's metadata, but not the code > > > itself. > > > >>>>> > > > > >>>>> > > > > >>>>> > > > > >>>>> > > > > >>>>> > On Sat, Mar 2, 2019 at 12:42 AM Thomas Weise < > > t...@apache.org > > > >>>>> <mailto:t...@apache.org>> wrote: > > > >>>>> > > > > >>>>> > > It would be good to encourage participation of > > > non-committers > > > >>>>> in the > > > >>>>> > review > > > >>>>> > > process, so +1 for allowing everyone to operate the > bot. > > > >>>>> > > > > > >>>>> > > Github approval will show a green checkmark for > committer > > > >>>> approval > > > >>>>> > > (assuming accounts were linked via gitbox) - that > should > > > >> provide > > > >>>>> > sufficient > > > >>>>> > > orientation? > > > >>>>> > > > > > >>>>> > > I just noticed that flinkbot seems to act as Robert > when > > it > > > >>>>> comes to > > > >>>>> > label > > > >>>>> > > management? I think that is confusing (besides earning > > > Robert > > > >>>>> a lot of > > > >>>>> > > extra github notification mail thanks to participation > on > > > >>>>> every PR :) > > > >>>>> > > > > > >>>>> > > Overall flinkbot is very useful, thanks for all the > work > > on > > > >>>>> it! I heard > > > >>>>> > > positive feedback from other contributors, I think they > > see > > > >> their > > > >>>>> > > contributions are better received now. > > > >>>>> > > > > > >>>>> > > Thomas > > > >>>>> > > > > > >>>>> > > > > > >>>>> > > > > > >>>>> > > On Fri, Mar 1, 2019 at 8:38 AM Robert Metzger > > > >>>>> <rmetz...@apache.org <mailto:rmetz...@apache.org>> > > > >>>>> > wrote: > > > >>>>> > > > > > >>>>> > > > I will update labels only based on committer's > > approvals > > > >> (for > > > >>>>> > > everything), > > > >>>>> > > > I think that's cleaner. > > > >>>>> > > > > > > >>>>> > > > We can always revisit this. > > > >>>>> > > > > > > >>>>> > > > On Wed, Feb 27, 2019 at 4:31 PM Chesnay Schepler > > > >>>>> <ches...@apache.org <mailto:ches...@apache.org>> > > > >>>>> > > > wrote: > > > >>>>> > > > > > > >>>>> > > > > Fore code-quality/description I agree, but > consensus > > > and > > > >>>>> the final > > > >>>>> > > > > approval should require a committer IMO. > > > >>>>> > > > > > > > >>>>> > > > > On 27.02.2019 15:08, Robert Metzger wrote: > > > >>>>> > > > > > > > >>>>> > > > > I did not put any restrictions on who can > communicate > > > with > > > >>>>> the bot! > > > >>>>> > > > > But since there is currently no way of overriding > > > >>>>> somebody's approval > > > >>>>> > > for > > > >>>>> > > > > something, this can easily lead to such a > situation. > > > >>>>> > > > > > > > >>>>> > > > > My thinking was that a committer still needs to > > > manually > > > >>>>> check who > > > >>>>> > > > > approved a pull request, and I wanted to be open > for > > > >>>>> non-committers > > > >>>>> > to > > > >>>>> > > > > participate in the review process. > > > >>>>> > > > > WIth the labels in place, this can easily send the > > > wrong > > > >>>>> message. > > > >>>>> > > > > > > > >>>>> > > > > What should we do? > > > >>>>> > > > > A) we restrict sending commands to the bot to > > > committers? > > > >>>>> > > > > B) only approvals from committers matter for > applying > > > >> labels? > > > >>>>> > > > > C) we allow committers to override approvals > > > >>>>> > > > > > > > >>>>> > > > > I'm leaning towards B, as it encourages > > non-committers > > > to > > > >>>>> > participate. > > > >>>>> > > > > > > > >>>>> > > > > > > > >>>>> > > > > On Wed, Feb 27, 2019 at 2:01 PM Chesnay Schepler > > > >>>>> <ches...@apache.org <mailto:ches...@apache.org> > > > >>>>> > > > > > >>>>> > > > > wrote: > > > >>>>> > > > > > > > >>>>> > > > >> Just noticed that _anyone_ can approve a PR now, > see > > > >>>>> > > > >> https://github.com/apache/flink/pull/7801. > > > >>>>> > > > >> > > > >>>>> > > > >> Not sure about the solution, but as it stands it > is > > > >>>>> rather trivial > > > >>>>> > to > > > >>>>> > > > >> nuke the review process of the entire project. > > > >>>>> > > > >> > > > >>>>> > > > >> On 13.02.2019 10:29, Robert Metzger wrote: > > > >>>>> > > > >> > Hey all, > > > >>>>> > > > >> > > > > >>>>> > > > >> > the flinkbot has been active for a week now, > and I > > > hope > > > >>>> the > > > >>>>> > initial > > > >>>>> > > > >> hiccups > > > >>>>> > > > >> > have been resolved :) > > > >>>>> > > > >> > > > > >>>>> > > > >> > I wanted to start this as a permanent thread to > > > discuss > > > >>>>> problems > > > >>>>> > and > > > >>>>> > > > >> > improvements with the bot. > > > >>>>> > > > >> > > > > >>>>> > > > >> > *So please post here if you have questions, > > > problems or > > > >>>>> ideas how > > > >>>>> > to > > > >>>>> > > > >> > improve it!* > > > >>>>> > > > >> > > > > >>>>> > > > >> > > > >>>>> > > > >> > > > >>>>> > > > > > > > >>>>> > > > > > > >>>>> > > > > > >>>>> > > > > >>>>> > > > >> > > > > > > > > >