Skip to content

fix: add deser and myblob to cspell word list to fix Spell check CI#10702

Draft
Copilot wants to merge 2 commits into
xmlperffrom
copilot/fix-spell-check-job
Draft

fix: add deser and myblob to cspell word list to fix Spell check CI#10702
Copilot wants to merge 2 commits into
xmlperffrom
copilot/fix-spell-check-job

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 15, 2026

Summary

The "Spell check" GitHub Actions job (check run ID 76222486900) was failing because cspell flagged two unknown words found in http-client-python source files:

  • deser — used in packages/http-client-python/generator/pygen/codegen/serializers/model_serializer.py and model_base.py.jinja2
  • myblob — used in packages/http-client-python/tests/unit/test_model_base_xml_serialization.py

Changes

  • Added deser and myblob to the words list in cspell.yaml (in alphabetical order)

l0lawrence and others added 2 commits May 15, 2026 09:56
…eturn-statements

Refactor _get_xml_deserializer_name from a 15-branch isinstance chain
into a tuple-driven loop with special-case handlers for DatetimeType,
ByteArraySchema, and EnumType. Brings return count from 15 down to 5,
below pylint's default cap of 6. Behaviour is unchanged.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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