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

do not perform autodetection to reduce memory consuption in server #1784

Merged
merged 6 commits into from
Oct 29, 2024

Conversation

motatoes
Copy link
Contributor

@motatoes motatoes commented Oct 29, 2024

autodetection with absense of digger.yml is an undocumented feature and it causes alot of memory consumption during directory walking which happens as a result. We remove it in this PR to save on server memory

Copy link

delve-auditor bot commented Oct 29, 2024

No security or compliance issues detected. Reviewed everything up to a5621da.

Security Overview
  • 🔎 Scanned files: 2 changed file(s)
Detected Code Changes
Change Type Relevant files
Enhancement ► github.go
    Add handling for PushEvent
    Implement handlePushEvent function
Refactor ► digger_config.go
    Remove AutoDetectDiggerConfig function
    Update LoadDiggerConfigYaml function
Other ► digger_config_test.go
    Update tests to reflect changes in digger_config.go

Reply to this PR with @delve-auditor followed by a description of what change you want and we'll auto-submit a change to this PR to implement it.

@motatoes motatoes merged commit cb94001 into develop Oct 29, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants