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

🤖 Init update.yml workflow #160

Merged
merged 3 commits into from
Sep 26, 2024
Merged

Conversation

conda-bot
Copy link
Contributor

This adds the update.yml workflow (and the accompanying config.yml) to this repository to enable regular syncing and templating of standard files (e.g., Code of Conduct file, CLA bot workflow, stale bot workflow, etc.).

This PR was triggered by @kenodegard via workflow_dispatch.

Auto-generated by the init.yml workflow, see https://github.com/conda/infrastructure/actions/runs/11061066717.

@conda-bot conda-bot requested a review from a team as a code owner September 26, 2024 22:36
@@ -0,0 +1,50 @@
conda/governance:
# [required] community files
- CODE_OF_CONDUCT.md
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is all thats needed to resolve #145

conda/infrastructure:
# [required] general workflows
- .github/workflows/cla.yml
- .github/workflows/update.yml
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is required to keep all files populated by this workflow in sync

# [optional] standard PR template
# - src: templates/pull_requests/news_tests_docs.md
# dst: .github/template-files/templates/pull_request_template_details.md
- src: templates/pull_requests/base.md
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@beeankha As a heads-up, this might interfere with #158. I'm going to approve this anyways so we get the other bot goodies rolling out.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We might end up commenting out some of the optional sections here. I'd like to see what they do first.

Copy link
Contributor

@kenodegard kenodegard Sep 26, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://github.com/conda/conda-package-streaming is an example that syncs these issue/PR templates and doesn't make any further modifications

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok sweet. These do look nice but I would like some of these auto-labeled with some of the labels we created. Is there a way to patch these templates with some customization?

I'm definitely going to disable the epic issue though. In our project board, we're using milestones as swim lanes and personally I think that is a little redundant with the epics concept.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can see the templates in conda/infrastructure:

some of them include fields that can be templated:

https://github.com/conda/infrastructure/blob/c24a6b6e68f1a4cc50e4cee8d625cf6ffd9861cb/templates/HOW_WE_USE_GITHUB.md?plain=1#L20

[workflow-update]: [[ repo.html_url ]]/blob/main/.github/workflows/update.yml

others include the option to define additional templates to be injected:

https://github.com/conda/infrastructure/blob/c24a6b6e68f1a4cc50e4cee8d625cf6ffd9861cb/templates/issues/bug.yml#L36

[% include 'issue_template_details.yml' ignore missing %]

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

currently the templates do not offer additional control over the labels

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good to know, thanks for your help on this!

@kenodegard
Copy link
Contributor

@schuylermartin45 let me know if you have any questions about anything in this workflow/config

Copy link
Collaborator

@schuylermartin45 schuylermartin45 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I'm currently happy with the configurations I've modified. We can always iterate on this later. @beeankha for visibility

@schuylermartin45 schuylermartin45 merged commit c7879b1 into conda-incubator:main Sep 26, 2024
12 checks passed
@kenodegard
Copy link
Contributor

now that this is merged the updates will automatically occur once a week on Sundays or can be manually trigged in Actions

@schuylermartin45
Copy link
Collaborator

schuylermartin45 commented Sep 27, 2024

Running this now to get the first changes ahead of Monday. Figured I could do this, just got busy last night.

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

Successfully merging this pull request may close these issues.

3 participants