Feat/tox python 3.14

[vinitkumar]

No description provided.

To fix the issue, the unused import of CaptureFixture should be removed from the TYPE_CHECKING block. This will eliminate the unnecessary dependency and improve code readability. The change should be made directly in the tests/test_dict2xml.py file.

To fix the issue, the unused import of FixtureRequest should be removed from the TYPE_CHECKING block. This will eliminate the unnecessary dependency and make the code cleaner. The change is localized to the TYPE_CHECKING block, and no other modifications are required.

To fix the problem, we will remove the unused import statement from _pytest.logging import LogCaptureFixture on line 12. This will eliminate the unnecessary dependency and improve the clarity of the code. No other changes are required, as this import is not used anywhere in the file.

To fix the problem, we should remove the unused import statement for MonkeyPatch. This will clean up the code and eliminate the unnecessary dependency. The change should be made in the tests/test_dict2xml.py file, specifically on line 13.

---

To fix the problem, we should remove the unused import from pytest_mock.plugin import MockerFixture from the code. This will clean up the code and eliminate the unnecessary dependency. No other changes are required since the removal of this import does not affect the functionality of the code.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.56%. Comparing base (4fe2b66) to head (f7f9f71).
⚠️ Report is 13 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #240      +/-   ##
==========================================
+ Coverage   97.22%   97.56%   +0.33%     
==========================================
  Files           6        3       -3     
  Lines         722      288     -434     
==========================================
- Hits          702      281     -421     
+ Misses         20        7      -13     

Flag	Coverage Δ
unittests	97.56% <100.00%> (+0.33%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

To fix the issue, we will remove the redundant import import json2xml.dicttoxml as module on line 780. Instead, we will use the already imported dicttoxml module (line 7) to access and patch the required functions. This eliminates the duplication while preserving the functionality of the test.

To fix the issue, we will remove the second import (import json2xml.dicttoxml as module) and use the existing dicttoxml import to access the is_primitive_type function. This will eliminate the redundancy while maintaining the functionality of the test. Specifically:

1. Replace module.is_primitive_type with dicttoxml.is_primitive_type.
2. Remove the import json2xml.dicttoxml as module statement.

---

[Copilot]

Pull Request Overview

This PR adds support for Python 3.14 by updating the tox, pyproject, and GitHub Actions configuration while also improving type annotations and default parameter handling in the dicttoxml module.

• Updated tox.ini to include the new py314-full environment and additional configuration options
• Modified dicttoxml functions to enhance type support and avoid mutable defaults
• Adjusted CI workflows and testing configuration to target new Python versions

Reviewed Changes

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

Show a summary per file

File	Description
tox.ini	Added new build options for Python 3.14 and updated environment settings
tests/conftest.py	Added sample JSON fixtures for testing
pyproject.toml	Updated pytest configuration with new file patterns and options
json2xml/dicttoxml.py	Modified type annotations and default parameters in conversion functions
AGENT.md	Updated documentation for build and test commands
.github/workflows/pythonpackage.yml	Updated workflow triggers, caching strategy, and updated dependencies for the new Python 3.14 version

Comments suppressed due to low confidence (2)

json2xml/dicttoxml.py:557

• The parameter 'obj' in the dicttoxml function is now typed as ELEMENT, which may be ambiguous if ELEMENT is not clearly defined. Please ensure that ELEMENT is defined and documented to avoid confusion.

    obj: ELEMENT,

json2xml/dicttoxml.py:513

• The docstring for convert_kv should be updated to mention that the function also supports date objects (in addition to datetime, string, and number types) to accurately reflect its updated type signature.

    """Converts a number, string, or datetime into an XML element"""

[Copilot]

[nitpick] The union type for 'val' explicitly lists int and float alongside numbers.Number, which already covers these types. Consider removing the redundant types for clarity, unless the explicitness is needed.

Suggested change

	val: str | int | float | numbers.Number | datetime.datetime | datetime.date,
	val: str | numbers.Number | datetime.datetime | datetime.date,

To fix the issue, we will remove the unused variable result from the code. Since the assignment to result does not have any side effects and is not used elsewhere in the function, we can safely remove the left-hand side of the assignment while keeping the call to subprocess.run. This ensures that the function's behavior remains unchanged.

---