Mark Battersby markalfred

Yeah, the "container vs component" distinction is falling more and more out of favor with me. It's an okay pattern to start, but it's a pretty hard black/white in a place where things are a lot more grey.

In my opinion, there's no reason for this:

// app/index.jsx
import AppComponent from './App'

const AppContainer = ({foo, bar, baz, buz, wham, bam, alakazam, setFoo, setBar, setBaz, setBuz, setWham, setBam, setAlakazam}) =>
  <AppComponent foo={foo} bar={bar} baz={baz} buz={buz} wham={wham} bam={bam} alakazam={alakazam} setFoo={setFoo}, setBar={setBar}, setBaz={setBaz}, setBuz={setBuz}, setWham={setWham}, setBam={setBam}, setAlakazam={setAlakazam} />

The trouble with ES6 named import/exports

-or-

Regarding renaming `SET_RELATIONSHIP_INFORMATION_ERRORS` to `SET_ERRORS`:

I've noticed a particular issue that seems to have come about with the advent of named import/exports in ES6 — that is, that variable names have grown in length by orders of magnitude. This seems to be the result of the need to namespace. For instance, let's say you have a set of user actions...

// user-actions.js

	#!/bin/sh

	# Alot of these configs have been taken from the various places
	# on the web, most from here
	# https://github.com/mathiasbynens/dotfiles/blob/master/.osx

	# Set the colours you can use
	black='\033[0;30m'
	white='\033[0;37m'
	red='\033[0;31m'

	def determine_output_pane(self):
	"""
	When the user performs some action which results in a file being
	opened or generated in a new tab, we should place that "output"
	code in an adjacent pane. This method will determine the correct
	pane to place the output in.

	It will use a tab to the right, if one exists in the current row.
	If not, output is placed in the adjacent tab to the left. Output
	should always be adjacent, and should never be on a different row,

	# Find the longest words you can type, alternating hands at each character. Y and B excluded.

	egrep "^[$right]?([$left][$right]){6,}[$left]?\$" /usr/share/dict/words

	# Qwerty
	left=qwertasdfgzxcv
	right=uiophjklnm

	# Colemak
	left=qwfpgarstdzxcv

	<snippet>
	<content><![CDATA[
	Given(/^${1:Something called}${2: "(.*?)"}${3: does something}\$/) do \|${4:variable_name}\|
	${0:}
	end
	]]></content>
	<!-- Optional: Set a tabTrigger to define how to trigger the snippet -->
	<tabTrigger>given-variable step</tabTrigger>
	<!-- Optional: Set a scope to limit where the snippet will trigger -->
	<!-- <scope>source.python</scope> -->

Mark Battersby markalfred

The trouble with ES6 named import/exports

Regarding renaming SET_RELATIONSHIP_INFORMATION_ERRORS to SET_ERRORS:

Regarding renaming `SET_RELATIONSHIP_INFORMATION_ERRORS` to `SET_ERRORS`: