Skip to content

Refactoring #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 6 commits into from
Nov 9, 2021
Merged

Refactoring #2

merged 6 commits into from
Nov 9, 2021

Conversation

@XedinUnknown
Copy link
Member

@XedinUnknown XedinUnknown commented Nov 9, 2021 •
edited
Loading

  • BC Break: Drop support for PHP < 7.2, and add support for PHP >= 8.
  • BC Break: TransformerFactoryInterface#make() renamed to createTransformer().
    It is now standalone, and doesn't require params.
  • BC Break: Removed outdated tools and deps.
  • TransformerAwareInterface#getTransformer() is now required to return a transformer.
  • Cleaned up configuration.
  • Added PHPStorm config.
  • Upgraded PHPUnit.
  • Add meta and other project configs.
  • Refactored tests.
  • Add code quality tools.
  • Add Github Actions config.

@XedinUnknown
Refactoring
1402ca2 
- Drop support for PHP < 7.2, and add support for PHP >= 8.
- Cleaned up configuration.
- Added PHPStorm config.
- Removed outdated tools and deps.
- Upgraded PHPUnit.
- Add meta and other project configs.
- Refactored tests.
- Add code quality tools.
- Add Github Actions config.
@XedinUnknown XedinUnknown added the enhancement New feature or request label Nov 9, 2021
@XedinUnknown XedinUnknown added this to the v0.2 milestone Nov 9, 2021
@XedinUnknown XedinUnknown self-assigned this Nov 9, 2021
@XedinUnknown
Update changelog
3f147b8
@XedinUnknown
Further clean up tests
1cf05f9
@XedinUnknown
Factory no longer requires params
c0ab25a 
Any configuration that's necessary to be performed around consumption time can be done via `with*()` methods.
@XedinUnknown
Factory no longer requires params
d7e1b30 
Any configuration that's necessary to be performed around consumption time can be done via `with*()` methods.
@XedinUnknown XedinUnknown merged commit 13e7c29 into 0.2.x Nov 9, 2021
@XedinUnknown XedinUnknown deleted the task/update branch November 9, 2021 06:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mecha mecha Awaiting requested review from mecha

@strangerkir strangerkir Awaiting requested review from strangerkir

Assignees

@XedinUnknown XedinUnknown

Labels

enhancement New feature or request

Projects

None yet

Milestone

v0.2

Development

Successfully merging this pull request may close these issues.

1 participant

@XedinUnknown