In D612#9490, @alken-orin wrote:Build has FAILED
Link to build: https://ci.nasqueron.org/job/test-notifications-phab/159/
See console output for more information: https://ci.nasqueron.org/job/test-notifications-phab/159/console
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed Advanced Search
Advanced Search
Advanced Search
Aug 20 2016
Aug 20 2016
dereckson retitled D619: Fix .gitignore files in storage/app from to Fix .gitignore files in storage/app.
dereckson moved T853: Deploy a Let's encrypt certificate to the Mumble server from Backlog to Needs Review / Blocked / Waiting on the User-Dereckson board.
dereckson added a project to T991: Document notifications center configuration schemas: Technical debt.
dereckson moved T991: Document notifications center configuration schemas from Backlog to Next on the User-Dereckson board.
dereckson moved T953: Handle Jenkins notification plugin payloads from Backlog to In progress on the User-Dereckson board.
dereckson moved T826: Create a systemd timer to renew letsencrypt certificat on Dwellers from Backlog to In progress on the User-Sandlayth board.
dereckson added a project to T295: Create a Vim default system configuration on Ysul: User-Sandlayth.
dereckson added a project to T694: See how to organize storage/ folder regeneration: User-Sandlayth.
dereckson added a project to T769: Note the base image of every container launched in the Agora table : User-Sandlayth.
Next step is to provide a configuration with:
- the mapping job/project
- emulate what we would need at Jenkins plugin level: only fire notifications on job failed
dereckson committed rNOTIF8d700507fd7c: Add support for Jenkins notification plug-in (authored by dereckson).
Add support for Jenkins notification plug-in
Rebased. More doc.
@Rashk0 any advice?
Aug 19 2016
Aug 19 2016
Summary of previous discussions
dereckson moved T843: Handle GitHub pull_request event from Backlog to Next on the User-Dereckson board.
dereckson moved T844: Handle GitHub issue_comment event from Backlog to Next on the User-Dereckson board.
dereckson added a project to D618: Fix docker run command's MySQL environment in readme: github-pull-request.
dereckson committed rDPHAB6813590b6237: Fix docker run command's MySQL environment in readme (authored by Gaurav Arora <gauravsworld@gmail.com>).
Fix docker run command's MySQL environment in readme
dereckson closed D618: Fix docker run command's MySQL environment in readme by committing rDPHAB6813590b6237: Fix docker run command's MySQL environment in readme (authored by Gaurav Arora <gauravsworld@gmail.com>).
Submitted by Gaurav Arora at https://github.com/nasqueron/docker-phabricator/pull/4
dereckson retitled D618: Fix docker run command's MySQL environment in readme from to Fix docker run command's MySQL environment in readme.
dereckson moved T891: .+prenom/.+nom: check if item already exists from Next to Backlog on the User-Dereckson board.
dereckson moved T984: Pick a format for Notifications::$rawContent property from Backlog to Next on the User-Dereckson board.
dereckson added a project to T984: Pick a format for Notifications::$rawContent property: User-Dereckson.
Add notifications:payload command
dereckson closed T983: CLI command to get a notification payload from a service payload as Resolved by committing rNOTIFb42260e53626: Add notifications:payload command.
Better code documentation
In D617#9435, @dereckson wrote:There is an issue in logic code too.
[ErrorException] Undefined property: stdClass::$repository
Payload was already a notification, not a DockerHub payload.
More tests.
There is an issue in logic code too.
dereckson retitled D617: Add notifications:payload command from to Add notifications:payload command.
Clean commented out code
dereckson closed D526: Clean commented out code by committing rDWD92ff7cf74e2f: Clean commented out code.
Avoid shadow variable name
dereckson closed D569: Avoid shadow variable name by committing rDWDa991d5d09084: Avoid shadow variable name.
Aug 18 2016
Aug 18 2016
dereckson added inline comments to rDPHAB82a577551716: Merge 48b2ffbf1481dbab46100e6d5bf9999ec34826df into….
dereckson committed rDPHABf1d02969c688: Improve code readability: 8 Mb, restart operation (authored by xcombelle).
Improve code readability: 8 Mb, restart operation
dereckson closed D602: Improve code readability: 8 Mb, restart operation by committing rDPHABf1d02969c688: Improve code readability: 8 Mb, restart operation (authored by xcombelle).
Rebased against master.
That would be set -e
There is no demonstrated advantage to use that, the json parsing issue in D347 was a false issue.
dereckson awarded D615: Explain why mysql.host is written in hard a Piece of Eight token.
dereckson committed rGROVEb3ae033adfc2: Define explicitly method visibility (authored by dereckson).
Define explicitly method visibility
dereckson retitled D616: Define explicitly method visibility from to Define explicitly method visibility.
According Bradipo from Freenode tcl channel, parsing is okay (and produces a correct dict) with json 1.3.3.
JSON makes a lot of sense, there is no need to parse twice a JSON payload, once for our notification format, once for the source payload.
Aug 17 2016
Aug 17 2016
dereckson committed rKBROKER545d2db54f4c: Explicitely set visibility to public methods (authored by dereckson).
Explicitely set visibility to public methods
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator