Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

@alloy => Convert back to static libs #832

Merged
merged 5 commits into from
Oct 1, 2015
Merged

@alloy => Convert back to static libs #832

merged 5 commits into from
Oct 1, 2015

Conversation

orta
Copy link
Contributor

@orta orta commented Sep 30, 2015

Re: #586

So far, I've only converted app-code. This runs 👍 - we have no actual need for using frameworks yet other than keeping up-to-date. So they are definitely not worth the current launch time.

@orta
Copy link
Contributor Author

orta commented Sep 30, 2015

seeing as you cleaned out the issues. I added some new ones for you.

screen shot 2015-09-30 at 18 41 53

@alloy
Copy link
Contributor

alloy commented Oct 1, 2015

Great, thanks! Can you fix the warnings so I can merge it?

@orta
Copy link
Contributor Author

orta commented Oct 1, 2015

Be glad I like you, that's an interns job. Done.

Giphy

@alloy
Copy link
Contributor

alloy commented Oct 1, 2015

Delivering PRs without warnings, though… 😝

@orta orta mentioned this pull request Oct 1, 2015
3 tasks
@alloy
Copy link
Contributor

alloy commented Oct 1, 2015

Based on the make ci build output, it appears there are still like 4 warnings.

@orta
Copy link
Contributor Author

orta commented Oct 1, 2015

ah yeah, tests

@orta
Copy link
Contributor Author

orta commented Oct 1, 2015

I think circle just went down, but I pushed a commit for the the four ; in tests

@alloy
Copy link
Contributor

alloy commented Oct 1, 2015

Thanks!

alloy added a commit that referenced this pull request Oct 1, 2015
@alloy => Convert back to static libs
@alloy alloy merged commit 74accfe into master Oct 1, 2015
@orta orta deleted the orta-statics branch December 14, 2015 22:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants