pratikmallya/autolabeler-codeowners

✨🏷️ Github Action to add labels to PRs based on CODEOWNERS

autolabeler-codeowners's Issues

File not found: master/lib/main.js

Hey @pratikmallya

This looks like it could be a really useful action :) I tried to set it up on one of my repos, but when running it, I get the following error:

Error: File not found: '/home/runner/work/_actions/pratikmallya/autolabeler-codeowners/master/lib/main.js'

Is this something that I am doing wrong, or maybe a bug?

For reference, here is the build log and my action YAML file. I've double-checked that my token is correct, and my .github/CODEOWNERS file is present.

Thanks very much :)
Alicia

---

Process all changed files rather than just the first 100

https://octokit.github.io/rest.js/#usage currently it processes only the first 100 changed files (max page size). Change this to get all the files.

Add automated e2e testing

Every merge to master should kick off a set of integration tests that would:

• open a PR on a fake repo with codeowners (maybe pratikmallya/test? Or just create a new dedicated repo for this)

• check that the action creates labels as expected

  • should label with existing labels for existing codeowners
  • should create new labels for new codeowners

• choose what framework/tools to use for this testing

  • would be nice if there's an existing framework that allows for this sort of testing

Add retries and throttling

seems simple enough: https://octokit.github.io/rest.js/#usage

provide demo PR

i notice this repo doesn't have a CODEOWNERS file, nor can i find a PR that demonstrates how the labels would appear.

perhaps even just some additions to the README that illustrate what people could expect.