seanreece / pr-reporter-slack Goto Github PK

It would be good to have a marker for stale PR's, and perhaps have this marker configurable (nbr of days).

Currently, latest tag v1.0.7 contains unescaped single quote in action.yml, hence it is unusable.
Also README.md suggests using v1 tag, but there is no v1 tag.

I'm seeing this error with v1.0.7. Same issue is not present in v1.0.6

Download action repository 'SeanReece/[email protected]' (SHA:2d3a7c13df7549f4706a90b17629e7ac2a95f868)
Error: SeanReece/pr-reporter-slack/v1.0.7/action.yml:
Error: SeanReece/pr-reporter-slack/v1.0.7/action.yml: (Line: 17, Col: 35, Idx: 588) - (Line: 17, Col: 47, Idx: 600): While parsing a block mapping, did not find expected key.
Error: System.ArgumentException: Unexpected type '' encountered while reading 'action manifest root'. The type 'MappingToken' was expected.
   at GitHub.DistributedTask.ObjectTemplating.Tokens.TemplateTokenExtensions.AssertMapping(TemplateToken value, String objectDescription)
   at GitHub.Runner.Worker.ActionManifestManager.Load(IExecutionContext executionContext, String manifestFile)
Error: Fail to load SeanReece/pr-reporter-slack/v1.0.7/action.yml

We use [WIP] in the PR title to mark that a PR is work in progress, but I'm sure other repos have different conventions. We should allow users to configure what flag they use to mark a work in progress PR.

Something like

inputs:
  wip-tag:
    description: 'The string used in PR titles to mark a Work-In-Progress PR'
    required: false
    default: '[WIP]'

Currently, pr-reporter-slack ignores draft PRs and PRs marked with [WIP]. Should allow user to configure this to message about ALL PRs

Maybe:

inputs:
  ignore-wip:
    description: 'Should pr-reporter-slack ignore work-in-progress PRs'
    required: false
    default: true

Currently we don't handle singular values correctly in the PR reporter message wording:

Example

SeanReece/pr-reporter-slack has 1 PRs ready for review

should be:

SeanReece/pr-reporter-slack has 1 PR ready for review

👉Fix some stuff | 1 approvals | 1 day ago

should be

👉Fix some stuff | 1 approval | 1 day ago

Github's dependabot can create a lot of PRs, should add a config to allow ignoring dependabot.

inputs:
  ignore-dependabot:
    description: 'Should pr-reporter-slack ignore dependabot PRs'
    required: false
    default: false

Hi! We love your action and use it on our team to check weekdaily for PRs. However, starting yesterday, the action started failing with an HTTP 400 Bad Request. We started using a self-hosted runner yesterday, which was my first thought, but it ran before we made the switch.

Here's our workflow file:

Here's the output:

There are no logs in the runner that indicate an error, just that the job was started.

Any troubleshooting instructions would be very well received. Thank you so much!

Currently pr-reporter-slack will send still send a message when there are no PRs awaiting review with something like "👍Good work! There's no PRs awaiting review"

Allow user to disable this for their repo. This would be helpful for smaller repos with less frequent PRs.

inputs:
  send-empty:
    description: 'Should send message when there are no PRs awaiting review'
    required: false
    default: true