Skip to content
This repository has been archived by the owner on Oct 4, 2020. It is now read-only.

Updates for 0.7 #20

Merged
merged 4 commits into from
Jul 3, 2015
Merged

Updates for 0.7 #20

merged 4 commits into from
Jul 3, 2015

Conversation

paf31
Copy link
Contributor

@paf31 paf31 commented Jul 2, 2015

@joneshf Could you please review this? Also, what do you think about moving to Pulp for the build?

@paf31
Copy link
Contributor Author

paf31 commented Jul 2, 2015

I still need to port the examples.

@paf31
Copy link
Contributor Author

paf31 commented Jul 2, 2015

The examples seemed broken and used some stuff from -index etc., but I created a small test module out of the Nested example, which was the easiest one to get running (for me).

@joneshf
Copy link
Contributor

joneshf commented Jul 3, 2015

Thanks!

I've not looked at pulp for quite a while. Does it support running user defined commands, and watching files? If I'd still need something like grunt/gulp/make for that I'd rather just stick to one build system.

Yeah, I think those examples rotted. Which reminds me that I wanted to have travis running for this thing.

joneshf added a commit that referenced this pull request Jul 3, 2015
@joneshf joneshf merged commit deff1b2 into master Jul 3, 2015
@paf31 paf31 deleted the 0.7-updates branch July 3, 2015 18:24
@paf31
Copy link
Contributor Author

paf31 commented Jul 3, 2015

In the core libraries, we've started using pulp build etc. as npm scripts, but I'm not sure if that supports watch or not.

When this is ready, could you please make a release? I'd like to update thermite-todomvc which uses lens.

Also, you might like to add the library to Pursuit 2 (I might send another PR soon to add some markdown comments if that's ok).

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.

None yet

2 participants