Skip to content

Conversation

@guardrex
Copy link
Collaborator

@guardrex guardrex commented Oct 25, 2024

@adegeo (cc: @Rick-Anderson @tdykstra @wadepickett) ... I'll give this a shot and see what happens ...

  • Moving the Blazor issue processing over to a new workflow.
  • The new workflow will apply the aspnet-core/svc and blazor/subsvc labels, along with the usual job tasks that I do for Blazor issues: place a 'hello' comment, label it with Blazor, assign me to the issue, and remove the Not Triaged label (if present ... if Repoman ran and applied it).
  • I'll try to run this workflow on an opened issue with a path of aspnetcore/blazor/**. I'm not sure about the on conditional checks because GH docs don't show an example of this combination working ... but we'll see soon enough! 😆

If this works, my recommendation will be to remove the initial labels that Repoman is applying and apply all them via a workflow.

I'll ping everyone in a moment from a test issue, and we can see if this actually works 🤞🍀.

If paths: - 'aspnetcore/blazor/**' doesn't trigger the job because paths only works for file push actions, then I'll try a conditional-based approach at the job level to see if I can get it to recognize the path text aspnet/core/blazor in the opening comment text.


Internal previews

📄 File 🔗 Preview link
.github/workflows/blazor-issue-processing.yml .github/workflows/blazor-issue-processing
.github/workflows/issue-processing.yml .github/workflows/issue-processing

@guardrex guardrex merged commit 1ef7de1 into main Oct 25, 2024
2 checks passed
@guardrex guardrex deleted the guardrex-patch-3 branch October 25, 2024 11:25
@guardrex
Copy link
Collaborator Author

guardrex commented Oct 25, 2024

I haven't tested yet; but the more I think about this, the less likely I think it will work. The path wouldn't make sense for issues ... only file changes on the repo (pull requests and pushes).

I'll move to trying an conditional job after this (probably) fails. I still have a chance to make a check on the opening issue comment text.

@wadepickett wadepickett mentioned this pull request Oct 25, 2024
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.

2 participants