Skip to content

Allow sending diagnostic logs to host server#3962

Merged
Withalion merged 8 commits intomasterfrom
send-logs
Jul 24, 2025
Merged

Allow sending diagnostic logs to host server#3962
Withalion merged 8 commits intomasterfrom
send-logs

Conversation

@ValentinBuira
Copy link
Copy Markdown
Contributor

fix #3885

Comment thread core/merginapi.cpp Outdated
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 2, 2025

Pull Request Test Coverage Report for Build 16047187598

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 392 unchanged lines in 3 files lost coverage.
  • Overall coverage decreased (-0.09%) to 59.542%

Files with Coverage Reduction New Missed Lines %
input/core/merginuserauth.cpp 9 64.36%
input/app/inputhelp.cpp 37 34.75%
input/core/merginapi.cpp 346 75.03%
Totals Coverage Status
Change from base Build 15995446192: -0.09%
Covered Lines: 8240
Relevant Lines: 13839

💛 - Coveralls

@ValentinBuira ValentinBuira marked this pull request as ready for review July 2, 2025 14:21
Copy link
Copy Markdown
Contributor

@Withalion Withalion left a comment

Choose a reason for hiding this comment

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

Looks good, just some minor changes

Comment thread app/inputhelp.cpp
Comment thread app/inputhelp.cpp Outdated
@tomasMizera tomasMizera added the FROZEN 🥶 do not merge before upcoming release label Jul 4, 2025
@tomasMizera tomasMizera removed the FROZEN 🥶 do not merge before upcoming release label Jul 21, 2025
@Withalion Withalion merged commit a430479 into master Jul 24, 2025
9 checks passed
@Withalion Withalion deleted the send-logs branch July 24, 2025 19:33
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.

Send diagnotic logs to the configured server

4 participants