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

[WIP] Adding Autotick bot docs to conda-forge main docs #1500

Closed
wants to merge 21 commits into from

Conversation

ForgottenProgramme
Copy link
Contributor

@ForgottenProgramme ForgottenProgramme commented Aug 22, 2021

Fixes #1460

maintainer documentation:

The stuff for the migration yaml docs is a lot less complete.

migration yaml docs:

@ForgottenProgramme ForgottenProgramme requested a review from a team as a code owner August 22, 2021 09:34
@ForgottenProgramme
Copy link
Contributor Author

@viniciusdc

@ForgottenProgramme
Copy link
Contributor Author

ForgottenProgramme commented Aug 22, 2021

bot_docs

@viniciusdc
Copy link
Contributor

viniciusdc commented Sep 3, 2021

Hi @ForgottenProgramme, some interesting bot's related issues can be found you Gitter if you want some examples of probematic situatuations. I will read thru the sections description later on today and add some comments in there if needed. Thanks for oppening this PR! Also don't forget to add the contact information list... eg. check most common response... comment on gitter, calll conda-forge bot team ...

@ForgottenProgramme
Copy link
Contributor Author

@beckermr

command for adding automerge (https://conda-forge.org/docs/maintainer/infrastructure.html#conda-forge-admin-please-add-bot-automerge)

I find this section a little confusing.

@beckermr
Copy link
Member

What's confusing? Happy to review a PR with any changes you have of course!

@ForgottenProgramme
Copy link
Contributor Author

@beckermr
Hey, Matt! :)

Please see this: https://conda-forge.org/docs/maintainer/infrastructure.html#conda-forge-admin-please-add-bot-automerge

This sentence:

...will instruct the admin bot to open a PR enabling the automatic merging of passing PRs from the auto-tick bot.

I don't understand this --"enabling the automatic merging of passing PRs from the auto-tick bot"

Also the link mentioned in that section no more exists.

@beckermr
Copy link
Member

Feel free to fix the link to point to point to the new docs here.

That text just means the command will generate a PR that will enabled automerge.

@ForgottenProgramme
Copy link
Contributor Author

ForgottenProgramme commented Sep 25, 2021

Feel free to fix the link to point to point to the new docs here.

Okay!

That text just means the command will generate a PR that will enabled automerge.

So does this work the same as the following key:

bot:
automerge: true

?

@beckermr
Copy link
Member

That command generates a PR on the feedstock with that change. You have to merge the pr. Feel free to try it out!

@ForgottenProgramme
Copy link
Contributor Author

Ahh.. got it now!
Thanks, Matt! :)

@viniciusdc
Copy link
Contributor

Hi, @ForgottenProgramme, could you update the status of this issue once you have some time?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Integrating the autotick bot docs with the conda-forge main docs
4 participants