Skip to content
This repository has been archived by the owner on Apr 25, 2018. It is now read-only.

Embrace Sanctuary #23

Merged
merged 1 commit into from
Mar 14, 2017
Merged

Embrace Sanctuary #23

merged 1 commit into from
Mar 14, 2017

Conversation

Avaq
Copy link
Owner

@Avaq Avaq commented Feb 24, 2017

Use sanctuary and sanctuary-def over Ramda, to promote a safe paradigm of writing code.

The bulk of changes is represented by the addition of the following two files:

  • src/env.js - Creates an env for Sanctuary
  • src/prelude.js - Instantiates Sanctuary and extends it with some common functions

@davidchambers, you might find it interesting to see how I plan on using Sanctuary in real life. The usage displayed here influences the way I think about approaching some of the problems in Sanctuary, like my solutions for sanctuary-js/sanctuary#327 (comment)

⚠️ This PR depends on sanctuary-js/sanctuary#354 being merged and released!

src/env.js Outdated
+ 'Reference/Global_Objects/Array/sort#Description';

const last = xs => xs[0];
const is = (T, x) => test(exports.env, T, x);
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the only reason I'm waiting for sanctuary-js/sanctuary#354, and I might as well pass [] instead of env, but I prefer to "do it right" from the start.

@davidchambers
Copy link

davidchambers commented Feb 26, 2017

This PR depends on sanctuary-js/sanctuary#354 being merged and released!

Done!

@Avaq Avaq force-pushed the embrace-sanctuary branch from 6b57a9c to c30802b Compare February 28, 2017 09:38
@Avaq Avaq merged commit cbe0e4a into master Mar 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants