Skip to content

Switch between getchar() and _getch() in windows when input is injected#139

Merged
rsanchez15 merged 3 commits into
mainfrom
fix/fix_fastddss_ci_windows
Jul 14, 2025
Merged

Switch between getchar() and _getch() in windows when input is injected#139
rsanchez15 merged 3 commits into
mainfrom
fix/fix_fastddss_ci_windows

Conversation

@depink5

@depink5 depink5 commented Jul 10, 2025

Copy link
Copy Markdown
Contributor

No description provided.

…ed through stdin

Signed-off-by: Denisa <denisa.alexandru@alumnos.upm.es>
@depink5 depink5 marked this pull request as draft July 10, 2025 15:23
Signed-off-by: Denisa <denisa.alexandru@alumnos.upm.es>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 62.17%. Comparing base (89c1650) to head (a7c6cee).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #139      +/-   ##
==========================================
- Coverage   62.96%   62.17%   -0.80%     
==========================================
  Files          65       65              
  Lines        1974     1864     -110     
  Branches      589      535      -54     
==========================================
- Hits         1243     1159      -84     
+ Misses        393      384       -9     
+ Partials      338      321      -17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Denisa <denisa@eprosima.com>
@rsanchez15 rsanchez15 merged commit 3bfcc19 into main Jul 14, 2025
38 of 41 checks passed
@rsanchez15 rsanchez15 deleted the fix/fix_fastddss_ci_windows branch July 14, 2025 12:26
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.

3 participants