search

influxdata / influxdb-php

influxdb-php: A PHP Client for InfluxDB, a time series database
http://influxdb.com
MIT License
432 stars 125 forks source link

Auto update on packagist #45

Open gianarb opened 8 years ago

gianarb commented 8 years ago

Hello @thecodeassassin do you have permission to add the packagist's token in order to sync in automatic our update on packagist?

@virgofx thanks for your ping

Thanks a lot PS at the moment the 1.4.0 is not up to date on packagist

virgofx commented 8 years ago

:+1:

virgofx commented 8 years ago

ping @thecodeassassin @gianarb

Any way we can get this auto-updated (still having to use custom repository until packagist updates). Just need to get the token from Packagist and add it in the admin settings/webhooks for this Repo. There is a Packagist section (super easy)

thecodeassassin commented 8 years ago

@virgofx i am currently on vacation but will take care of this as soon as possible. sorry for the late reply

thecodeassassin commented 8 years ago

Autoupdate was already enabled, i refreshed the token (maybe that was causing a problem).

virgofx commented 8 years ago

Weird, still shows only the 1.4 version. Must be something going on. I believe you need to have both the correct username and token from your Packagist profile. (The domain isn't necessary) Ensure the github.com setting also has enabled. Then head back to packagist and maybe try to update ...

image

image

thecodeassassin commented 8 years ago

@virgofx exactly, i have no idea why it doesn't work. The github service says the payload was delivered successfully :S

tuupola commented 8 years ago

While you are at it, try clicking the green update button in Packagist. I am keen to test the new version.

tuupola commented 8 years ago

What are you using as username in GitHub Packagist hook settings?

thecodeassassin commented 8 years ago

@tuupola the exact username i use on packagist.

thecodeassassin commented 8 years ago

@tuupola updated

tuupola commented 8 years ago

Try changing the GitHub repository address in Packagist to current one. Currently Packagist has https://github.com/influxdb/influxdb-php but it has been changed at some point to https://github.com/influxdata/influxdb-php. Maybe this confuses Packagist.

virgofx commented 8 years ago

Also the composer.json needs to have the name updated as well. New packages don't work unless they match properly.

virgofx commented 8 years ago

I would leave existing packagist as is. Then add a newone with proper name, marked as deprecated in readme. This will give people time to switch composer updates.

tuupola commented 8 years ago

Also the composer.json needs to have the name updated as well. New packages don't work unless they match properly.

This is not true.

The package name has not been changed. Only repository address has been changed. Package name and repository address do not need to match.

Note that I was not talking about package name. I am talking about repository address in Packagist settings.

screenshot_139

You can change it by clicking the blue edit button and entering new repository address.

screenshot_140

My guess is that webhook comes from different repository than what is in Packagist settings and this causes Packagist to ignore the incoming webhook.

thatside commented 6 years ago

Hello!

Please consider changing package name on Packagist according to new Github name (see https://github.com/composer/packagist/issues/47). Current naming doesn't allow to use forks of this repo as a replacement of master repo for dependant packages (like https://github.com/Algatux/influxdb-bundle). These packages depend on influxdb/influxdb-php, but forked packages automatically become influxdata/influxdb-php based on composer.json info.