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

Créer un package sur Packagist #5

Closed
ghost opened this issue Jun 13, 2013 · 4 comments
Closed

Créer un package sur Packagist #5

ghost opened this issue Jun 13, 2013 · 4 comments

Comments

@ghost
Copy link

ghost commented Jun 13, 2013

Salut Etienne,

Tout d'abord merci beaucoup pour ce code que tu mets à disposition de tous et met à jour régulièrement, c'est vraiment pratique !

J'ai juste une proposition à faire: est-ce que tu pourrais mettre ton repo Github sur https://packagist.org/ ?

C'est utile pour pouvoir intégrer l'api allociné helper dans des projets utilisant le gestionnaire de dépendances Composer.
C'est vraiment facile à faire et ça prend deux secondes et cela permet de toujours avoir la dernière version sans avoir à tout re-télecharger sur Github.

Merci beaucoup en tout cas !

@etn406
Copy link
Owner

etn406 commented Jun 15, 2013

Salut,
J'ai bien ajouté tout ça sur Packagist, peux-tu me dire si ça fonctionne ?
J'ai l'impression qu'il y a un problème au niveau des mises à jour automatiques, et pourtant j'ai bien fait ce qui était dit.
https://packagist.org/packages/api-allocine/api-allocine-helper

EDIT : L'erreur a disparu. J'imagine que c'était le temps que la page se mette à jour :)

@ghost
Copy link
Author

ghost commented Jun 16, 2013

Salut,

Merci beaucoup pour avoir mis tout ça sur packagist ! C'est vraiment pratique maintenant.
Deux petites remarques cependant:

  • Normalement la ligne a ajouté devrait finir par "dev-master", c'est comme ça pour tous les autres paquets. Pour toi, vu que la branche principale sur GitHub s'appelle "php" et non "master" ça donne une fin de ligne en "dev-php".
    C'est juste un point de vue subjectif mais le mieux serait de renommer la branche principale en "master" pour que le paquet sur Packagist soit plus 'conventionnel'.
  • Il faut automatiser le chargement de tes classes via composer en modifiant le fichier composer.json:
    Regarde en dessous pour voir le contenu que je propose. Tu remarqueras que j'ai renommé le fichier principal pour retirer la version à la fin. C'est plus propre comme ça. Si tu veux préciser la version, mets la en commentaire au début du fichier.

En tout cas merci beaucoup pour ta réponse rapide, ça fait plaisir :)

{
    "name": "api-allocine/api-allocine-helper",
    "description": "An easy way to use the Allocine (and its other foreign websites) API",
    "type": "library",
    "homepage": "https://github.com/etienne-gauvin/api-allocine-helper",
    "license": "CC-BY-SA",
    "authors": [
        {
            "name": "Etienne Gauvin",
            "email": "etiennegauvin41@gmail.com"
        }
    ],
    "require": {
        "php": ">=5.3.0"
    },
    "autoload": {
        "files": ["api-allocine-helper.php"]
    },
}

@etn406
Copy link
Owner

etn406 commented Jun 17, 2013

J'ai suivi tes conseils, merci d'avoir rédigé le fichier composer.json :) !
Tout semble ok, dev-php a bien été remplacé par dev-master.

@ghost
Copy link
Author

ghost commented Jun 17, 2013

Nickel, merci beaucoup !
Je ferme l'issue :)

@ghost ghost closed this as completed Jun 17, 2013
@blg81 blg81 mentioned this issue Jul 14, 2020
This issue was closed.
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

No branches or pull requests

1 participant