{"action":"created","issue":{"url":"https://api.github.com/repos/patacrep/patacrep/issues/176","labels_url":"https://api.github.com/repos/patacrep/patacrep/issues/176/labels{/name}","comments_url":"https://api.github.com/repos/patacrep/patacrep/issues/176/comments","events_url":"https://api.github.com/repos/patacrep/patacrep/issues/176/events","html_url":"https://github.com/patacrep/patacrep/pull/176","id":116945293,"number":176,"title":"Add an `errors` attribute to songs, compiling the song errors","user":{"login":"paternal","id":6587757,"avatar_url":"https://avatars.githubusercontent.com/u/6587757?v=3","gravatar_id":"","url":"https://api.github.com/users/paternal","html_url":"https://github.com/paternal","followers_url":"https://api.github.com/users/paternal/followers","following_url":"https://api.github.com/users/paternal/following{/other_user}","gists_url":"https://api.github.com/users/paternal/gists{/gist_id}","starred_url":"https://api.github.com/users/paternal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/paternal/subscriptions","organizations_url":"https://api.github.com/users/paternal/orgs","repos_url":"https://api.github.com/users/paternal/repos","events_url":"https://api.github.com/users/paternal/events{/privacy}","received_events_url":"https://api.github.com/users/paternal/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/patacrep/patacrep/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/patacrep/patacrep/labels/pr:work%20in%20progress","name":"pr:work in progress","color":"fbca04"},{"url":"https://api.github.com/repos/patacrep/patacrep/labels/song","name":"song","color":"fad8c7"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-11-14T18:57:17Z","updated_at":"2015-11-21T01:24:18Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/patacrep/patacrep/pulls/176","html_url":"https://github.com/patacrep/patacrep/pull/176","diff_url":"https://github.com/patacrep/patacrep/pull/176.diff","patch_url":"https://github.com/patacrep/patacrep/pull/176.patch"},"body":"Closes #121 and #123."},"comment":{"url":"https://api.github.com/repos/patacrep/patacrep/issues/comments/158571946","html_url":"https://github.com/patacrep/patacrep/pull/176#issuecomment-158571946","issue_url":"https://api.github.com/repos/patacrep/patacrep/issues/176","id":158571946,"user":{"login":"paternal","id":6587757,"avatar_url":"https://avatars.githubusercontent.com/u/6587757?v=3","gravatar_id":"","url":"https://api.github.com/users/paternal","html_url":"https://github.com/paternal","followers_url":"https://api.github.com/users/paternal/followers","following_url":"https://api.github.com/users/paternal/following{/other_user}","gists_url":"https://api.github.com/users/paternal/gists{/gist_id}","starred_url":"https://api.github.com/users/paternal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/paternal/subscriptions","organizations_url":"https://api.github.com/users/paternal/orgs","repos_url":"https://api.github.com/users/paternal/repos","events_url":"https://api.github.com/users/paternal/events{/privacy}","received_events_url":"https://api.github.com/users/paternal/received_events","type":"User","site_admin":false},"created_at":"2015-11-21T01:24:18Z","updated_at":"2015-11-21T01:24:18Z","body":"> Est-ce qu'il existe des erreurs propres au carnet ? (ou est-ce que c'est pour avoir toutes les erreurs des chants du carnet ?)\r\n\r\nC'est la compilation des erreurs des chants. Dans l'immédiat, je ne vois pas d'erreur propre au carnet (y aurait-il un intérêt à y ajouter les erreurs de compilation LaTeX ? Je ne suis pas sûr).\r\n\r\n> Ca me fait penser que différents niveaux d'erreur pourraient être intéressant (erreur fatale ou erreur de langue ne devraient pas être traitées de la même manière\r\n\r\nPour moi, la seule erreur fatale qu'on a (bloc non fermé) est une erreur du parseur, pas du chant (ça ne devrait pas planter autant).\r\n\r\n> On pourrait séparer les deux (et rajouter une option --consistencyerror=pass) qui vérifie que les \"pièces jointes\" sont bien présentes par exemple\r\n\r\nDans le code, c'est [déjà prévu](https://github.com/patacrep/patacrep/commit/f8c84de29c14a0627b2e75f04121953ff3659261#diff-4fef879cfba8a01ac1d8cd5c79dbfd63R29). Comme ça, les programmes appelant patacrep (patanet et patagui) peuvent en faire ce qu'ils veulent. Pour l'interface en ligne de commande, je ne pense pas que ce soit nécessaire.\r\n\r\n> Je préfère que l'argument de error indique que faire des erreurs, plutôt que quand l'erreur doit être remontée.\r\n\r\nSauf que j'ai l'impression que c'est nécessaire : avec ton argument `gather`, on collecte les erreurs, mais est-ce que qu'on continue ou est-ce qu'on s'arrête (avant ou après la compilation latex).\r\n\r\nJ'ai l'impression qu'on va avoir du mal à avoir quelque chose de cohérent : l'affichage des erreurs et les ignorer répondent à la question « que faire des erreurs », alors que l'arrêt après les erreurs d'un chant ou d'un carnet répondent à la question « quand arrêter en cas d'erreur ». Ou alors on utilise des options looooongues : `--onerror=failonsong/failonbook/ignore/warn`"}}