Skip to content

Conversation

bernhardkaindl
Copy link
Contributor

Fix spelling errors and Markdown errors in docs/README-Unicode.md

  • For GitHub Copilot PR review: There is no xcp.python, xcp.compat is the correction.

Included Base commit for CI:

  • Update CI to pass coverage (fails currently due to a 0.1% calculation change).
  • remove the temporary exclusion of xcp/dmv.py from the diff/patch-coverage analysis

A simple PR without product code changes to fix CI and spelling/Markdown errors.

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Updates CI configuration to fix coverage thresholds and removes temporary exclusions, while fixing spelling and Markdown formatting errors in the Unicode documentation.

  • Adjusts CI coverage requirements from 96% to 95% and removes temporary exclusion of xcp/dmv.py
  • Corrects multiple spelling errors and improves grammar in the Unicode documentation
  • Adds exclusion for auto-generated Markdown files in pre-commit linting

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
tox.ini Updates coverage thresholds and removes temporary file exclusion
README-Unicode.md Fixes spelling errors, grammar issues, and Markdown formatting
.pre-commit-config.yaml Excludes auto-generated files from Markdown linting

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@bernhardkaindl bernhardkaindl force-pushed the README-Unicode.md-Fix-spelling-and-markdownlint branch from d62b6f4 to 9fbeb5b Compare September 23, 2025 08:16
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

codecov bot commented Sep 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@          Coverage Diff           @@
##           master    #179   +/-   ##
======================================
  Coverage    83.3%   83.3%           
======================================
  Files          23      23           
  Lines        3347    3347           
======================================
  Hits         2790    2790           
  Misses        557     557           

Signed-off-by: Bernhard Kaindl <bernhard.kaindl@cloud.com>
Signed-off-by: Bernhard Kaindl <bernhard.kaindl@cloud.com>
@bernhardkaindl bernhardkaindl force-pushed the README-Unicode.md-Fix-spelling-and-markdownlint branch from 9fbeb5b to fa89fdb Compare September 23, 2025 08:56
@bernhardkaindl
Copy link
Contributor Author

Superceded by the cleaned-up #182

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.

1 participant