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

Change Crunz vendor #506

Merged
merged 2 commits into from
Jun 5, 2022
Merged

Change Crunz vendor #506

merged 2 commits into from
Jun 5, 2022

Conversation

PabloKowalczyk
Copy link
Contributor

Crunz changed repository: lavary/crunz#411

@turtle0x1
Copy link
Owner

Thanks very much for this!

Symfony carries some weight in the community (I.E they can ask Github), have you tried every avenue to rescue the 1400 star repo you have? Changing over to a very low star repo seems like it could cause unnecessary concern.

Using git diff its clear you haven't poisoned the repo but i'm "scared" to merge this change.

Thanks

@turtle0x1
Copy link
Owner

Well @fabpot the leader of symfony project has accepted this so... I will as well.

@turtle0x1 turtle0x1 merged commit 5182123 into turtle0x1:master Jun 5, 2022
@turtle0x1
Copy link
Owner

For those interested, here is the link symfony/marketing#547

@PabloKowalczyk PabloKowalczyk deleted the new-crunz branch June 6, 2022 07:05
@PabloKowalczyk
Copy link
Contributor Author

Hello,

Symfony carries some weight in the community (I.E they can ask Github), have you tried every avenue to rescue the 1400 star repo you have?

Actually it's not my repo this is why I can't do much there. Tried to contact @lavary but without response.

Using git diff its clear you haven't poisoned the repo but i'm "scared" to merge this change.

That's understandable and all I can do is to advise installing https://github.com/pyrech/composer-changelogs to show links to diff after every update, with this you can quickly check every dependency changes.

Also, I was maintainer of https://github.com/lavary/crunz, now I focus on https://github.com/crunzphp/crunz, so only repo name changed :)

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