Work with babel

[fundon]

Hi, currently I write ES6+ code and use Babel transform them.

Maybe we can include it and add an option for it.

Like this below:

# lib/gulpfile.js

"use strict";

require("babel/register");

var gulp    = require("gulp");
var path    = require("path");
var fs      = require("fs");
var helpers = require(path.resolve(__dirname, "helpers"));

require("gulp-help")(gulp, {
  aliases: ["h"],
  afterPrintCallback: helpers.gulp.printManuals
});

...

If you like this, I can submit a PR.

[sdepold]

While I don't really need ES6 for the CLI at this point, I wonder if the end-user will benefit from such a change?

[fundon]

I refactor my project and use ES6+ by Babel.
The database.js below and the migrations/*.js files will do that.
So I need babel to transform code before task execute.

export default {

  "development": {
    "url": process.env.DATABASE_URL,
    "username": "trek",
    "password": "star trek",
    "database": "trek",
    "host": "192.168.59.103",
    "port": "5432",
    "dialect": "postgres",
    "native": true,
    "timezone": "+00:00",

    "query": {
      "pool": true,
      "debug": true
    },

    "define": {
      "timestamps": true,
      "underscored": true,
      "charset": "utf8"
    },
    "logging": true
  },

  "test": {
    "url": process.env.DATABASE_URL,
    "username": "root",
    "password": null,
    "database": "database_test",
    "host": "127.0.0.1",
    "dialect": "postgres"
  },

  "production": {
    "url": process.env.DATABASE_URL,
    "username": process.env.DATABASE_USERNAME,
    "password": process.env.DATABASE_PASSWORD,
    "database": process.env.DATABASE_DATABASE,
    "host": process.env.DATABASE_HOST,
    "port": process.env.DATABASE_PORT,
    "dialect": "postgres"
  }

};

[fundon]

BTW, likes the --coffee flag.

[xpepermint]

+1

[xpepermint]

@sdepold Any progress on this? My projects are all es6 (babel). What do you suggest?

[xpepermint]

Btw... I think it would be enough if you just add require("babel/register"); into lib/gulpfile.js and bin/sequelize. I made a fork https://github.com/xpepermint/cli just to test that and it works but I think you should check if that's true.

[ngs]

+1

[nkt]

@sdepold there are a lot of benefits. Look at this:

async function up(query, Sequelize) {
  await query.createTable('users', ...);
  await query.createIndex('users', ...);
}

[nkt]

Simple advise: move your sequelize configuration info separated file, where you can require babel hook. That's how it looks like in my project:

// src/config/sequelize.js
require('babel-core/register');
const config = require('./index').database;

module.exports = {
  development: config,
  test: config,
  production: config
};

[xpepermint]

Any progress on that? @sdepold?

[fundon]

I closed it.

[xpepermint]

@fundon You say that is works? The example below doesn't work and I still have to use my patch https://github.com/xpepermint/cli.

export default {
  up(migration, Sequelize) {
    return migration.createTable('Projects', {
      id: {allowNull: false, autoIncrement: true, primaryKey: true, type: Sequelize.INTEGER},
      name: {type: Sequelize.STRING}
    });
  },
  down(migration, Sequelize) {
    return migration.dropTable('Projects');
  }
};

[xpepermint]

Ow... you mean this #113? Doesn't work. Can you please point me to the docs?