Skip to content

[-] fix assignment to entry in nil map, fixes #968#969

Merged
pashagolub merged 1 commit intomasterfrom
968_fix_nil_map_access
Sep 22, 2025
Merged

[-] fix assignment to entry in nil map, fixes #968#969
pashagolub merged 1 commit intomasterfrom
968_fix_nil_map_access

Conversation

@pashagolub
Copy link
Copy Markdown
Collaborator

Add NewSourceConn() constructor and use direct access in FetchRuntimeInfo() without temp RuntimeInfo var

Add `NewSourceConn()` constructor and use direct access in
`FetchRuntimeInfo()` without temp `RuntimeInfo` var
@pashagolub pashagolub self-assigned this Sep 22, 2025
@pashagolub pashagolub added the bug Something isn't working label Sep 22, 2025
@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 17917333314

Details

  • 19 of 20 (95.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 70.366%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/sources/resolver.go 1 2 50.0%
Totals Coverage Status
Change from base Build 17915631207: 0.02%
Covered Lines: 3519
Relevant Lines: 5001

💛 - Coveralls

@pashagolub pashagolub merged commit 982a9a2 into master Sep 22, 2025
9 checks passed
@pashagolub pashagolub deleted the 968_fix_nil_map_access branch September 22, 2025 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants