feat(logs): add a new log when an issue consumed at least one operation #386

C0ZEN · 2021-03-10T21:40:53Z

Changes:

Add one log at the end of the issue process to display the count of consumed operations (only if at least one was consumed)
Add a better documentation for operations-per-run option
Make the issue extend the operations
Split in half the operations to separate the logic of consuming some operations and doing that plus with a counter (useful to avoid coding the same logic twice)
Disable chalk colours for the tests to avoid to test the sugar it provides

Closes #348

feat(stale-and-close): add new options to change the days before clos…

Remove the stale label when labeled with an exempt one (actions#268)

feat(statistics): display some stats in the logs (actions#337)

* docs(only-labels): remove duplicated option and improve descriptions a bad rebase happend * docs(readme): use a multi-line array and remove the optional column the option column was not helpful since each value is optional the multi-line array will allow to have a better UI in small devices and basically in GitHub too due to the max-width * style(readme): break line for the statistics * docs(readme): add a better description for the ascending option * docs(action): add missing punctuation

Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 4.15.2 to 4.16.1. - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v4.16.1/packages/eslint-plugin) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [@octokit/rest](https://github.com/octokit/rest.js) from 18.3.0 to 18.3.2. - [Release notes](https://github.com/octokit/rest.js/releases) - [Commits](octokit/rest.js@v18.3.0...v18.3.2) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

the log will be visible as the last row of the processing of the given issue closes actions#348

HonkingGoose

Some suggestions, if you'd like them. 😉

README.md

Co-authored-by: HonkingGoose <[email protected]>

C0ZEN · 2021-03-18T21:06:57Z

@HonkingGoose each time the action needs to interact with the GitHub API to alter the project content (CRUD) and to process the issues and PRs like adding the stale label, at some point on huge projects and based also on the other actions present in the project you could hit the GitHub API limit.
So in order to have a "limit" for one to avoid consuming all the possible API calls and also to let's say "set up a viable limit" this option was created.
The default value is very low for each kind of project IMO and obviously the "right" limit is totally dependent of the project and the stale config.
The CRON frequency should be set IMO to run one time per day, it's the more logical setup.
Obviously having a better doc is a thing and this is one of my primary goal for the next PR (once all my current one are landed).

README.md

HonkingGoose · 2021-03-19T09:58:20Z

@C0ZEN Thank you for your answer! 😄

Co-authored-by: HonkingGoose <[email protected]>

C0ZEN

Merge LGTM

Not tests but feels like a typo. Or if we keep the title it `exempt` feels more approriate: exempt-issue-labels: 'roadmap' exempt-pr-labels: 'roadmap'

…ctions#380) * docs(only-labels): enhance the docs and fix duplicate (actions#341) * docs(only-labels): remove duplicated option and improve descriptions a bad rebase happend * docs(readme): use a multi-line array and remove the optional column the option column was not helpful since each value is optional the multi-line array will allow to have a better UI in small devices and basically in GitHub too due to the max-width * style(readme): break line for the statistics * docs(readme): add a better description for the ascending option * docs(action): add missing punctuation * build(deps-dev): bump @typescript-eslint/eslint-plugin (actions#342) Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 4.15.2 to 4.16.1. - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v4.16.1/packages/eslint-plugin) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump @octokit/rest from 18.3.0 to 18.3.2 (actions#350) Bumps [@octokit/rest](https://github.com/octokit/rest.js) from 18.3.0 to 18.3.2. - [Release notes](https://github.com/octokit/rest.js/releases) - [Commits](octokit/rest.js@v18.3.0...v18.3.2) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (actions#352) (#15) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (actions#352) (#17) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (actions#352) (#18) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * feat(any-of-labels): add 2 new options to customize for issues/PRs closes actions#371 change this option and only-labels to have tree-logs * chore(index): update it Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* docs(only-labels): enhance the docs and fix duplicate (actions#341) * docs(only-labels): remove duplicated option and improve descriptions a bad rebase happend * docs(readme): use a multi-line array and remove the optional column the option column was not helpful since each value is optional the multi-line array will allow to have a better UI in small devices and basically in GitHub too due to the max-width * style(readme): break line for the statistics * docs(readme): add a better description for the ascending option * docs(action): add missing punctuation * build(deps-dev): bump @typescript-eslint/eslint-plugin (actions#342) Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 4.15.2 to 4.16.1. - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v4.16.1/packages/eslint-plugin) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump @octokit/rest from 18.3.0 to 18.3.2 (actions#350) Bumps [@octokit/rest](https://github.com/octokit/rest.js) from 18.3.0 to 18.3.2. - [Release notes](https://github.com/octokit/rest.js/releases) - [Commits](octokit/rest.js@v18.3.0...v18.3.2) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (actions#352) (#15) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (actions#352) (#17) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (actions#352) (#18) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * feat(logs): enhance the logs for assignees and milestones closes actions#381 Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

src/classes/issues-processor.ts

C0ZEN · 2021-04-28T21:51:28Z

Merge LGTM

C0ZEN

Merge LGTM

luketomlinson

@C0ZEN I see what you are trying to do here, but I am going to have to do a little more thinking about this one. It seems there is a decent amount of complexity being added to be able to log operations per issue. I'm just wondering if there is a way to simplify, maybe with a simpler counter inside the loop for each issue? This might be easier to understand than making Issue extend Operations Thoughts?

src/classes/issues-processor.ts

src/classes/stale-operations.ts

src/main.ts

C0ZEN · 2021-04-30T23:55:34Z

@luketomlinson feedbacks handled.
I agree that there is some code repetition about the counters and it's not perfect.
On the other hand, having a count of operations performed for a given issue makes sense to me, so the point about Issue extend Operations is OKish for me.
I propose this instead though: don't extend, just expose an operations property 4cf12ba.

Here is also a slightly more reusable version which reduce the code duplication of the count for the issue. 96bee75

luketomlinson

Hey @C0ZEN I like the clean up! 💯

src/classes/issues-processor.ts

…on (actions#386) * docs(only-labels): enhance the docs and fix duplicate (actions#341) * docs(only-labels): remove duplicated option and improve descriptions a bad rebase happend * docs(readme): use a multi-line array and remove the optional column the option column was not helpful since each value is optional the multi-line array will allow to have a better UI in small devices and basically in GitHub too due to the max-width * style(readme): break line for the statistics * docs(readme): add a better description for the ascending option * docs(action): add missing punctuation * build(deps-dev): bump @typescript-eslint/eslint-plugin (actions#342) Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 4.15.2 to 4.16.1. - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v4.16.1/packages/eslint-plugin) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump @octokit/rest from 18.3.0 to 18.3.2 (actions#350) Bumps [@octokit/rest](https://github.com/octokit/rest.js) from 18.3.0 to 18.3.2. - [Release notes](https://github.com/octokit/rest.js/releases) - [Commits](octokit/rest.js@v18.3.0...v18.3.2) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (actions#352) (#15) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (actions#352) (#17) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (actions#352) (#18) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * docs(operations-per-run): improve the doc for this option * feat(logs): add a new log when an issue consumed at least one operation the log will be visible as the last row of the processing of the given issue closes actions#348 * chore(readme): improve the operations per run Co-authored-by: HonkingGoose <[email protected]> * chore(readme): improve the operations per run Co-authored-by: HonkingGoose <[email protected]> * chore(readme): improve the operations per run Co-authored-by: HonkingGoose <[email protected]> * chore(readme): improve the operations per run Co-authored-by: HonkingGoose <[email protected]> * chore(readme): improve the operations per run Co-authored-by: HonkingGoose <[email protected]> * chore(readme): improve the operations per run Co-authored-by: HonkingGoose <[email protected]> * Typo in how to perform check for specific labels (actions#357) Not tests but feels like a typo. Or if we keep the title it `exempt` feels more approriate: exempt-issue-labels: 'roadmap' exempt-pr-labels: 'roadmap' * feat(any-of-labels): add 2 new options to customize for issues/PRs (actions#380) * docs(only-labels): enhance the docs and fix duplicate (actions#341) * docs(only-labels): remove duplicated option and improve descriptions a bad rebase happend * docs(readme): use a multi-line array and remove the optional column the option column was not helpful since each value is optional the multi-line array will allow to have a better UI in small devices and basically in GitHub too due to the max-width * style(readme): break line for the statistics * docs(readme): add a better description for the ascending option * docs(action): add missing punctuation * build(deps-dev): bump @typescript-eslint/eslint-plugin (actions#342) Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 4.15.2 to 4.16.1. - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v4.16.1/packages/eslint-plugin) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump @octokit/rest from 18.3.0 to 18.3.2 (actions#350) Bumps [@octokit/rest](https://github.com/octokit/rest.js) from 18.3.0 to 18.3.2. - [Release notes](https://github.com/octokit/rest.js/releases) - [Commits](octokit/rest.js@v18.3.0...v18.3.2) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (actions#352) (#15) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (actions#352) (#17) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (actions#352) (#18) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * feat(any-of-labels): add 2 new options to customize for issues/PRs closes actions#371 change this option and only-labels to have tree-logs * chore(index): update it Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * feat(logs): enhance the logs for assignees and milestones (actions#382) * docs(only-labels): enhance the docs and fix duplicate (actions#341) * docs(only-labels): remove duplicated option and improve descriptions a bad rebase happend * docs(readme): use a multi-line array and remove the optional column the option column was not helpful since each value is optional the multi-line array will allow to have a better UI in small devices and basically in GitHub too due to the max-width * style(readme): break line for the statistics * docs(readme): add a better description for the ascending option * docs(action): add missing punctuation * build(deps-dev): bump @typescript-eslint/eslint-plugin (actions#342) Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 4.15.2 to 4.16.1. - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v4.16.1/packages/eslint-plugin) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump @octokit/rest from 18.3.0 to 18.3.2 (actions#350) Bumps [@octokit/rest](https://github.com/octokit/rest.js) from 18.3.0 to 18.3.2. - [Release notes](https://github.com/octokit/rest.js/releases) - [Commits](octokit/rest.js@v18.3.0...v18.3.2) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (actions#352) (#15) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (actions#352) (#17) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (actions#352) (#18) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * feat(logs): enhance the logs for assignees and milestones closes actions#381 Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * style(typo): fix typo plural issue * style(naming): rename two methods * chore(error): remove a potential useless throw of error * style(naming): rename one method * refactor(issue): change the way to count the operations * refactor(operations): create a method to reduce code duplication Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: HonkingGoose <[email protected]> Co-authored-by: Romain Rigaux <[email protected]>

* feat(logs): add a new log when an issue consumed at least one operation (#386) * docs(only-labels): enhance the docs and fix duplicate (#341) * docs(only-labels): remove duplicated option and improve descriptions a bad rebase happend * docs(readme): use a multi-line array and remove the optional column the option column was not helpful since each value is optional the multi-line array will allow to have a better UI in small devices and basically in GitHub too due to the max-width * style(readme): break line for the statistics * docs(readme): add a better description for the ascending option * docs(action): add missing punctuation * build(deps-dev): bump @typescript-eslint/eslint-plugin (#342) Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 4.15.2 to 4.16.1. - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v4.16.1/packages/eslint-plugin) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump @octokit/rest from 18.3.0 to 18.3.2 (#350) Bumps [@octokit/rest](https://github.com/octokit/rest.js) from 18.3.0 to 18.3.2. - [Release notes](https://github.com/octokit/rest.js/releases) - [Commits](octokit/rest.js@v18.3.0...v18.3.2) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (#352) (#15) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (#352) (#17) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (#352) (#18) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * docs(operations-per-run): improve the doc for this option * feat(logs): add a new log when an issue consumed at least one operation the log will be visible as the last row of the processing of the given issue closes #348 * chore(readme): improve the operations per run Co-authored-by: HonkingGoose <[email protected]> * chore(readme): improve the operations per run Co-authored-by: HonkingGoose <[email protected]> * chore(readme): improve the operations per run Co-authored-by: HonkingGoose <[email protected]> * chore(readme): improve the operations per run Co-authored-by: HonkingGoose <[email protected]> * chore(readme): improve the operations per run Co-authored-by: HonkingGoose <[email protected]> * chore(readme): improve the operations per run Co-authored-by: HonkingGoose <[email protected]> * Typo in how to perform check for specific labels (#357) Not tests but feels like a typo. Or if we keep the title it `exempt` feels more approriate: exempt-issue-labels: 'roadmap' exempt-pr-labels: 'roadmap' * feat(any-of-labels): add 2 new options to customize for issues/PRs (#380) * docs(only-labels): enhance the docs and fix duplicate (#341) * docs(only-labels): remove duplicated option and improve descriptions a bad rebase happend * docs(readme): use a multi-line array and remove the optional column the option column was not helpful since each value is optional the multi-line array will allow to have a better UI in small devices and basically in GitHub too due to the max-width * style(readme): break line for the statistics * docs(readme): add a better description for the ascending option * docs(action): add missing punctuation * build(deps-dev): bump @typescript-eslint/eslint-plugin (#342) Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 4.15.2 to 4.16.1. - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v4.16.1/packages/eslint-plugin) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump @octokit/rest from 18.3.0 to 18.3.2 (#350) Bumps [@octokit/rest](https://github.com/octokit/rest.js) from 18.3.0 to 18.3.2. - [Release notes](https://github.com/octokit/rest.js/releases) - [Commits](octokit/rest.js@v18.3.0...v18.3.2) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (#352) (#15) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (#352) (#17) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (#352) (#18) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * feat(any-of-labels): add 2 new options to customize for issues/PRs closes #371 change this option and only-labels to have tree-logs * chore(index): update it Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * feat(logs): enhance the logs for assignees and milestones (#382) * docs(only-labels): enhance the docs and fix duplicate (#341) * docs(only-labels): remove duplicated option and improve descriptions a bad rebase happend * docs(readme): use a multi-line array and remove the optional column the option column was not helpful since each value is optional the multi-line array will allow to have a better UI in small devices and basically in GitHub too due to the max-width * style(readme): break line for the statistics * docs(readme): add a better description for the ascending option * docs(action): add missing punctuation * build(deps-dev): bump @typescript-eslint/eslint-plugin (#342) Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 4.15.2 to 4.16.1. - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v4.16.1/packages/eslint-plugin) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump @octokit/rest from 18.3.0 to 18.3.2 (#350) Bumps [@octokit/rest](https://github.com/octokit/rest.js) from 18.3.0 to 18.3.2. - [Release notes](https://github.com/octokit/rest.js/releases) - [Commits](octokit/rest.js@v18.3.0...v18.3.2) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (#352) (#15) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (#352) (#17) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: add more coverage for the stale label behaviour (#352) (#18) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * feat(logs): enhance the logs for assignees and milestones closes #381 Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * style(typo): fix typo plural issue * style(naming): rename two methods * chore(error): remove a potential useless throw of error * style(naming): rename one method * refactor(issue): change the way to count the operations * refactor(operations): create a method to reduce code duplication Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: HonkingGoose <[email protected]> Co-authored-by: Romain Rigaux <[email protected]> * chore(logs): final overhaul of the logs display more values in cyan display all the options the same way (with links pointing to the readme so that with anchors we could find them easily later make them a little more humanized add more logs especially for the stale conditions * style(logs): fix typos Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: HonkingGoose <[email protected]> Co-authored-by: Romain Rigaux <[email protected]>

C0ZEN and others added 21 commits November 22, 2020 10:55

Merge pull request #1 from actions/main

c21e2f2

Merge pull request #2 from actions/main

c913fa4

Merge pull request #3 from actions/main

9cf9bf2

feat(stale-and-close): add new options to change the days before clos…

Merge pull request #4 from actions/main

f11f623

Merge pull request #5 from actions/main

0f1457a

Remove the stale label when labeled with an exempt one (actions#268)

Merge pull request #6 from actions/main

fef6333

Merge pull request #7 from actions/main

e1ba62c

Merge pull request #8 from actions/main

abab4a6

Merge pull request #9 from actions/main

920683c

Merge pull request #10 from actions/main

a98474b

Merge pull request #11 from actions/main

915ed27

Merge pull request #12 from actions/main

4b097b3

feat(statistics): display some stats in the logs (actions#337)

Merge pull request #14 from actions/main

5a8ec1e

test: add more coverage for the stale label behaviour (actions#352) (#15

6c29691

) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

test: add more coverage for the stale label behaviour (actions#352) (#17

d485d43

) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

test: add more coverage for the stale label behaviour (actions#352) (#18

25621f6

) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

docs(operations-per-run): improve the doc for this option

6257990

feat(logs): add a new log when an issue consumed at least one operation

c38af01

the log will be visible as the last row of the processing of the given issue closes actions#348

C0ZEN marked this pull request as ready for review March 10, 2021 21:41

Merge branch 'main' into feature/log-operations-per-issue

173c540

HonkingGoose reviewed Mar 16, 2021

View reviewed changes

README.md Outdated Show resolved Hide resolved

README.md Outdated Show resolved Hide resolved

README.md Outdated Show resolved Hide resolved

README.md Outdated Show resolved Hide resolved

README.md Outdated Show resolved Hide resolved

C0ZEN and others added 3 commits March 17, 2021 19:36

chore(readme): improve the operations per run

d0d072d

Co-authored-by: HonkingGoose <[email protected]>

chore(readme): improve the operations per run

6d7053e

Co-authored-by: HonkingGoose <[email protected]>

chore(readme): improve the operations per run

4236350

Co-authored-by: HonkingGoose <[email protected]>

C0ZEN force-pushed the feature/log-operations-per-issue branch from 21aa4c1 to 4236350 Compare March 17, 2021 18:38

C0ZEN and others added 2 commits March 17, 2021 19:38

chore(readme): improve the operations per run

e7b8a38

Co-authored-by: HonkingGoose <[email protected]>

chore(readme): improve the operations per run

304e352

Co-authored-by: HonkingGoose <[email protected]>

HonkingGoose reviewed Mar 19, 2021

View reviewed changes

README.md Outdated Show resolved Hide resolved

C0ZEN and others added 2 commits March 19, 2021 11:52

chore(readme): improve the operations per run

c78dbd5

Co-authored-by: HonkingGoose <[email protected]>

Merge branch 'main' into feature/log-operations-per-issue

9a29082

C0ZEN requested a review from a team as a code owner April 27, 2021 22:18

C0ZEN commented Apr 27, 2021

View reviewed changes

romainr and others added 5 commits April 28, 2021 23:30

Typo in how to perform check for specific labels (actions#357)

ee8f729

Not tests but feels like a typo. Or if we keep the title it `exempt` feels more approriate: exempt-issue-labels: 'roadmap' exempt-pr-labels: 'roadmap'

Merge branch 'main' into feature/log-operations-per-issue

8025ac6

Merge branch 'main' into feature/log-operations-per-issue

72abf2b

C0ZEN commented Apr 28, 2021

View reviewed changes

src/classes/issues-processor.ts Outdated Show resolved Hide resolved

style(typo): fix typo plural issue

2570e86

Merge branch 'main' into feature/log-operations-per-issue

ee986e5

C0ZEN commented Apr 30, 2021

View reviewed changes

luketomlinson reviewed Apr 30, 2021

View reviewed changes

src/classes/issues-processor.ts Outdated Show resolved Hide resolved

src/classes/stale-operations.ts Outdated Show resolved Hide resolved

src/classes/stale-operations.ts Outdated Show resolved Hide resolved

src/main.ts Show resolved Hide resolved

C0ZEN added 4 commits May 1, 2021 01:35

style(naming): rename two methods

385b1d6

chore(error): remove a potential useless throw of error

783c103

style(naming): rename one method

7fae5fa

refactor(issue): change the way to count the operations

4cf12ba

refactor(operations): create a method to reduce code duplication

96bee75

luketomlinson approved these changes May 3, 2021

View reviewed changes

src/classes/issues-processor.ts Show resolved Hide resolved

luketomlinson merged commit c11507e into actions:main May 3, 2021

C0ZEN deleted the feature/log-operations-per-issue branch May 3, 2021 15:49

C0ZEN mentioned this pull request May 5, 2021

feat(logs): add a new log when an issue consumed at least one operati… C0ZEN/stale#23

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(logs): add a new log when an issue consumed at least one operation #386

feat(logs): add a new log when an issue consumed at least one operation #386

C0ZEN commented Mar 10, 2021 •

edited

Loading

HonkingGoose left a comment

C0ZEN commented Mar 18, 2021

HonkingGoose commented Mar 19, 2021

C0ZEN left a comment •

edited

Loading

C0ZEN commented Apr 28, 2021

C0ZEN left a comment

luketomlinson left a comment

C0ZEN commented Apr 30, 2021 •

edited

Loading

luketomlinson left a comment

feat(logs): add a new log when an issue consumed at least one operation #386

feat(logs): add a new log when an issue consumed at least one operation #386

Conversation

C0ZEN commented Mar 10, 2021 • edited Loading

HonkingGoose left a comment

Choose a reason for hiding this comment

C0ZEN commented Mar 18, 2021

HonkingGoose commented Mar 19, 2021

C0ZEN left a comment • edited Loading

Choose a reason for hiding this comment

C0ZEN commented Apr 28, 2021

C0ZEN left a comment

Choose a reason for hiding this comment

luketomlinson left a comment

Choose a reason for hiding this comment

C0ZEN commented Apr 30, 2021 • edited Loading

luketomlinson left a comment

Choose a reason for hiding this comment

C0ZEN commented Mar 10, 2021 •

edited

Loading

C0ZEN left a comment •

edited

Loading

C0ZEN commented Apr 30, 2021 •

edited

Loading