Skip to content

Initial version#1

Merged
vitek-dev merged 1 commit intomainfrom
#1
Aug 4, 2025
Merged

Initial version#1
vitek-dev merged 1 commit intomainfrom
#1

Conversation

@vitek-dev
Copy link
Owner

No description provided.

@vitek-dev vitek-dev requested a review from Copilot August 4, 2025 14:11
@vitek-dev vitek-dev self-assigned this Aug 4, 2025
Copy link

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

This PR introduces the initial version of a JSON serializer library that provides a preconfigured Symfony serializer integration for the Nette Framework. The library enables serialization and deserialization of PHP objects to/from JSON with support for nested objects, arrays, enums, and discriminators.

  • Complete implementation of JsonSerializer class with serialize, deserialize, and decode methods
  • Nette DI extension for dependency injection integration
  • Comprehensive test suite covering basic data types, complex objects, and framework integration

Reviewed Changes

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

Show a summary per file
File Description
src/JsonSerializer.php Core serializer implementation with Symfony serializer configuration
src/DI/JsonSerializerExtension.php Nette DI extension for container integration
tests/SimpleTest.php Tests for basic data type serialization/deserialization
tests/SerializeTest.php Tests for complex object serialization
tests/DeserializeTest.php Tests for complex object deserialization
tests/DiTest.php Tests for dependency injection integration
tests/BaseTestCase.php Base test class with container bootstrapping
tests/Resources/*.php Test resource classes (Car, Owner, Garage, etc.)
Configuration files PHPUnit, PHPStan, PHPCS, Docker, Composer, and CI/CD setup
Comments suppressed due to low confidence (1)

tests/SerializeTest.php:45

  • The variable name 'companys' contains a grammatical error. It should be 'companyCar' or 'companysCar' to better reflect that it represents a company's car.
        $companys = $this->createCar(

@vitek-dev vitek-dev force-pushed the #1 branch 2 times, most recently from 4b9daa3 to 47dbee2 Compare August 4, 2025 14:15
@vitek-dev vitek-dev merged commit a9f173f into main Aug 4, 2025
4 checks passed
@vitek-dev vitek-dev deleted the #1 branch August 4, 2025 14:21
@vitek-dev vitek-dev restored the #1 branch August 4, 2025 14:22
@vitek-dev vitek-dev deleted the #1 branch August 4, 2025 14:23
vitek-dev added a commit that referenced this pull request Aug 4, 2025
vitek-dev added a commit that referenced this pull request Aug 4, 2025
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

Comments