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.

Sign up for GitHub

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

Jump to bottom

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

Merged
merged 19 commits into from
Mar 1, 2021
Merged

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

merged 19 commits into from
Mar 1, 2021

Conversation

C0ZEN
Copy link
Contributor

@C0ZEN C0ZEN commented Feb 27, 2021 •
edited
Loading

Changes:

  • Add enable-statistics option
  • Enhance the documentation regarding the debug
  • Change the way to mock the IssuesProcessor so that it no longers allow to override some methods from the constructor
  • Change the way the operations left are calculated to make sure it is made only when a GitHub operation is made (false positives and also missing operations) (only drawback is that the debug cancel before the count)

Closes #262

C0ZEN added 19 commits November 22, 2020 10:55
@C0ZEN
Merge pull request #1 from actions/main
c21e2f2
@C0ZEN
Merge pull request #2 from actions/main
c913fa4
@C0ZEN
Merge pull request #3 from actions/main
9cf9bf2 
feat(stale-and-close): add new options to change the days before clos…
@C0ZEN
Merge pull request #4 from actions/main
f11f623
@C0ZEN
Merge pull request #5 from actions/main
0f1457a 
Remove the stale label when labeled with an exempt one  (actions#268)
@C0ZEN
Merge pull request #6 from actions/main
fef6333
@C0ZEN
Merge pull request #7 from actions/main
e1ba62c
@C0ZEN
Merge pull request #8 from actions/main
abab4a6
@C0ZEN
Merge pull request #9 from actions/main
920683c
@C0ZEN
Merge pull request #10 from actions/main
a98474b
@C0ZEN
Merge pull request #11 from actions/main
915ed27
@C0ZEN
test: add more coverage
aa4be8e
@C0ZEN
docs: reorder and enhance typo
727b998
@C0ZEN
docs(contributing): add more information about the npm scripts
970e2e4
@C0ZEN
feat(statistics): add simple statistics
be26c01
@C0ZEN
feat(statistics): add more stats
5b50137
@C0ZEN
refactor(issues-processor): remove some options from the constructor
9e82add 
it should have been only useful for the tests
@C0ZEN
feat(statistics): add stats for new stale or undo stale issues
0a1c9be
@C0ZEN
chore(rebase): handle rebase conflicts
89a763c
@C0ZEN C0ZEN force-pushed the feature/statistics branch from 51eec36 to 89a763c Compare March 1, 2021 19:21
C0ZEN
C0ZEN commented Mar 1, 2021
View reviewed changes
Copy link
Contributor Author

@C0ZEN C0ZEN left a comment

Choose a reason for hiding this comment

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

@hross this one is ready and I think it's a good one to close the scope for the next release ;)

@hross hross merged commit 419a53b into actions:main Mar 1, 2021
@C0ZEN C0ZEN deleted the feature/statistics branch March 1, 2021 22:34
C0ZEN added a commit to C0ZEN/stale that referenced this pull request Mar 1, 2021
@C0ZEN
Merge pull request #12 from actions/main
4b097b3 
feat(statistics): display some stats in the logs (actions#337)
This was referenced Mar 6, 2021
Merged
Closed
Closed
Closed
Merged
Merged
This was referenced Mar 8, 2021
Closed
Closed
Closed
Merged
Closed
Merged
Closed
Closed
Closed
Closed
Merged
Closed
Merged
Closed
Closed
Merged
Closed
Closed
Merged
Merged
Merged
Merged
Merged
Merged
Closed
Closed
Closed
This was referenced Mar 16, 2021
Closed
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add statistics after run
2 participants
@C0ZEN @hross