kbendick commented on pull request #1289: URL: https://github.com/apache/iceberg/pull/1289#issuecomment-669090858
> > > @kbendick How could we verify the effect of this patch ? Fork my own iceberg repo and upload a spark test PR in my own repo, will it create a label for the PR ? It will be better if we have a verification. > > > Thanks for the nice work. > > > > > > To verify this patch, I first need to submit a few small changes to the configuration file (based on my misinterpretation of the format). > > Then, somebody with write access on this repository needs to enable the autolabeler bot by navigating to here https://github.com/apps/probot-autolabeler and then configuring the bot to have the permissions that it requests for this repository only. The permission set is relatively benign. I can link a screen shot if need be. This should be acceptable as the spark project is using it. > > Then, the labels do not need to be precreated. They will be generated when PRs are submitted. If one wants to pick the colors for the labels or add descriptions to them, this can be configured either by creating the label initially or by editing the labels on this page: https://github.com/apache/iceberg/issues/labels > > After I submit one more patch to the labeler yaml file, once it's merged it will start working as expected. I have demonstrated this in my own fork both here for an MR label [kbendick#7](https://github.com/kbendick/iceberg/pull/7) and here for a FLINK label [kbendick#8](https://github.com/kbendick/iceberg/pull/8). > > I much prefer them with color, but this can be dealt with after the initial set up is merged (and confirmed by others if they so wish). I will submit a final patch to make it into a working state and then allow time for any additional comments to the original configuration. > > @openinx for reference, the two labeled PRs that I linked were generated off of this configuration file in my personal fork. https://github.com/kbendick/iceberg/blob/master/.github/autolabeler.yml I have updated the configuration file such that it will work once somebody enables the bot / app https://github.com/apps/probot-autolabeler to have read permissions and write labels to PRs permissions (it's really self explanatory when clicking through it). ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
