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.

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

ci: make stalebot less annoying #1587

Merged
merged 2 commits into from
Jan 29, 2023
Merged

ci: make stalebot less annoying #1587

merged 2 commits into from
Jan 29, 2023

Conversation

catthehacker
Copy link
Member

No description provided.

@catthehacker catthehacker requested a review from a team as a code owner January 29, 2023 14:46
@github-actions
Copy link
Contributor

github-actions bot commented Jan 29, 2023

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ EDITORCONFIG editorconfig-checker 1 0 0.0s
✅ REPOSITORY gitleaks yes no 2.52s
✅ REPOSITORY git_diff yes no 0.0s
✅ REPOSITORY secretlint yes no 0.91s
✅ YAML prettier 1 0 0.39s
✅ YAML v8r 1 0 1.95s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@catthehacker catthehacker changed the title cI: make stalebot less annoying ci: make stalebot less annoying Jan 29, 2023
@codecov
Copy link

codecov bot commented Jan 29, 2023

Codecov Report

Merging #1587 (7c63d84) into master (4989f44) will increase coverage by 0.66%.
The diff coverage is 77.40%.

@@            Coverage Diff             @@
##           master    #1587      +/-   ##
==========================================
+ Coverage   61.22%   61.89%   +0.66%     
==========================================
  Files          46       46              
  Lines        7141     7185      +44     
==========================================
+ Hits         4372     4447      +75     
+ Misses       2462     2430      -32     
- Partials      307      308       +1     
Impacted Files Coverage Δ
pkg/container/docker_cli.go 82.23% <ø> (ø)
pkg/container/docker_logger.go 52.08% <ø> (ø)
pkg/container/docker_pull.go 33.33% <ø> (ø)
pkg/container/docker_run.go 13.58% <ø> (ø)
pkg/container/docker_volume.go 0.00% <ø> (ø)
pkg/runner/run_context.go 78.81% <0.00%> (ø)
pkg/container/docker_images.go 27.02% <29.41%> (-4.13%) ⬇️
pkg/container/docker_auth.go 51.35% <56.25%> (+3.73%) ⬆️
pkg/runner/runner.go 86.45% <66.66%> (-2.05%) ⬇️
pkg/common/git/git.go 57.72% <79.71%> (+7.72%) ⬆️
... and 10 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@ChristopherHX ChristopherHX left a comment

Choose a reason for hiding this comment

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

I would disable the stale bot. I don't really see a benifit in the stale bot at the moment, other than keeping the number of open issues < 100.

@catthehacker
Copy link
Member Author

I don't mind disabling it :P I think it's up to @cplee whether we should do that

@mergify mergify bot merged commit 01952d0 into master Jan 29, 2023
@mergify mergify bot deleted the stale-bot branch January 29, 2023 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants