Skip to content
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.

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

Remove laminas log, serializer and loader dependencies #65

Merged
merged 2 commits into from
Oct 12, 2021

Conversation

gsteel
Copy link
Member

@gsteel gsteel commented Feb 25, 2021

Q A
Documentation no
Bugfix no
BC Break no
New Feature no
RFC no
QA yes

Description

As per TSC #68, this pull removes the dev dependency on Laminas\Log

@gsteel gsteel changed the title Remove laminas-log dev dependency Remove laminas log, serializer and loader dependencies Feb 25, 2021
@weierophinney weierophinney added this to the 2.13.0 milestone Mar 1, 2021
@weierophinney
Copy link
Member

Please rebase to 2.13.x

@gsteel gsteel force-pushed the remove-laminas-log-dependency branch from 1d92f41 to 58ff4da Compare March 1, 2021 14:43
@weierophinney
Copy link
Member

Looks like errors on all the various checks, @gsteel — ping me when you have them sorted!

@gsteel
Copy link
Member Author

gsteel commented Mar 2, 2021

@weierophinney - This needs a rebase on 1.13.x 2.13.x when it's available, but I think that's blocked due to docs linting which I attempted in #72 but it's bit of a mare - I know nothing about docs build/formatting.

@weierophinney weierophinney changed the base branch from 2.12.x to 2.13.x October 12, 2021 17:08
@weierophinney weierophinney merged commit 8b8fc7f into laminas:2.13.x Oct 12, 2021
@gsteel gsteel deleted the remove-laminas-log-dependency branch October 12, 2021 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants