Skip to content

Conversation

@asgrim
Copy link
Contributor

@asgrim asgrim commented Sep 19, 2024

Adds support for PIE.

Once merged, a maintainer would need to add this repo's URL to https://packagist.org/packages/submit :)

More info/documentation can be read on: https://github.com/php/pie/blob/main/docs/extension-maintainers.md

@@ -0,0 +1,68 @@
{
"name": "php-memcached/php-memcached",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please verify vendor namespace & package name are acceptable

@m6w6 m6w6 merged commit 7227982 into php-memcached-dev:master Sep 26, 2024
@asgrim asgrim deleted the add-support-for-pie branch September 30, 2024 07:50
@TimWolla
Copy link

TimWolla commented Jan 2, 2025

Once merged, a maintainer would need to add this repo's URL to https://packagist.org/packages/submit :)

It appears this has not yet happened.

@sodabrew
Copy link
Collaborator

sodabrew commented Jan 2, 2025

Done! https://packagist.org/packages/php-memcached/php-memcached

@TimWolla
Copy link

TimWolla commented Jan 3, 2025

@sodabrew Thank you. I notice that Packagist says “This package is not auto-updated”. You would then need to remember to update the Packagist data for every release. It would probably make sense to also set up the webhook.

@asgrim
Copy link
Contributor Author

asgrim commented Jan 20, 2025

hi @sodabrew any chance the auto update could be set up as mentioned above? :) would certainly make things smoother for future releases :) thank you!

@asgrim
Copy link
Contributor Author

asgrim commented Jun 12, 2025

Ping on the above please! :)

@m6w6
Copy link
Contributor

m6w6 commented Jun 24, 2025

@sodabrew , since you own the package on packagist, could you please update the webhook?

@sodabrew
Copy link
Collaborator

@m6w6 done!

@TimWolla
Copy link

since you own the package on packagist

FYI: It's also possible to add additional maintainers to Packagist entry. The large “plus” in the screenshot should work.
image

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.

4 participants