nicgirault

Electrical comsumption in France in 2014 (with weekends and hollidays highlighted)

https://github.com/nicgirault/circosJS

nicgirault

resolve:
  updateDates: ($q, DataUpdate) ->
    def = $q.defer()
    DataUpdate.find()
    .$promise
    .then (updateDates) ->
      def.resolve updateDates
    .catch ->
      def.resolve []
    return def.promise

nicgirault

• le fichier se trouve au bon endroit
• le fichier porte le bon nom
• la variable est bien nommée
• la variable porte le bon typage (const, let, var)
• le code se trouve au bon endroit
• le code est testé
• les cas de tests sont suffisants
• il n'y a pas de duplication de code
• le code supprimé ne laisse pas de variables inutilisées, de module inutilisé, de dépendance inutilisée
• le code ne duplique pas une librairie connue

nicgirault

<script>
window['_fs_debug'] = false;
window['_fs_host'] = 'fullstory.com';
window['_fs_org'] = 'XXX';
window['_fs_namespace'] = 'FS';
(function(m,n,e,t,l,o,g,y){...})(window,document,window['_fs_namespace'],'script','user');
</script>

nicgirault

export const setFullStoryIdentity = (user) => {
  if (!window.FS) return
  
  window.FS.identify(user.id, {
    displayName: `${user.firstName} ${user.lastName}`,
    email: user.email
  })
}

nicgirault

import Raven from 'raven-js'
import config from './config'

if (config.RAVEN_ENDPOINT) {
  Raven.config(config.RAVEN_ENDPOINT).install()
}

nicgirault

elif [ "${CIRCLE_BRANCH}" == "master" ]; then
  export SENTRY_AUTH_TOKEN=xxx
  curl -sL https://sentry.io/get-cli/ | bash
  
  # `git rev-parse HEAD` return the hash of last commit and is used as release id
  RELEASE_ID=$(git rev-parse HEAD)
  
  # create new sentry release
  sentry-cli releases -o my-account -p my-app new $RELEASE_ID
  

nicgirault

{
  "scripts": {
    "build": "REACT_APP_RELEASE=$(git rev-parse HEAD) react-scripts build"
  }
}

nicgirault

import Raven from 'raven-js'
import config from './config'

if (config.RAVEN_ENDPOINT) {
  Raven.config(config.RAVEN_ENDPOINT, {release: process.env.REACT_APP_RELEASE}).install()
}

nicgirault

const ErrorStackParser = require('error-stack-parser')
const Slack = require('slack-node')

const config = require('../config')
const packageJson = require('../../package.json')

const slack = new Slack()
slack.setWebhook(config.slackHookUrl)

module.exports = (err, req, res, next) => {