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

Simplify Setup #1

Closed
westonganger opened this issue Sep 12, 2016 · 5 comments

Comments

Projects
None yet
2 participants
@westonganger
Copy link

commented Sep 12, 2016

This application has a complicated setup process for the development environment with foreman and nginx and other non-standard things. I feel like this is will hinder adoption of this gem. As far as I can tell this is for your own custom setup. Can we clean this up so it it uses more conventional setup like most normal rails apps.

@randy-girard

This comment has been minimized.

Copy link
Owner

commented Sep 12, 2016

I would agree that it does have a lot of non-standard dependencies, that we can filter out and simplify. I would say however, once having those, the setup should be fairly simple, as its just a bin/setup and bundle exec foreman start.

We can easily remove nginx, and probably just run puma or thin easily enough. There may need to be some work to move to Sqlite3 for some of the querying that is used and functions for date related parts.

I will work/look into simplifying these things.

@westonganger

This comment has been minimized.

Copy link
Author

commented Sep 12, 2016

Great to hear. Yeah it would really be good to be database agnostic for in my case I would prefer to use MySQL. Side question: Are you using this in production successfully currently?

@randy-girard

This comment has been minimized.

Copy link
Owner

commented Sep 12, 2016

I am not using it on any apps that I would consider production traffic. I have used it on apps that are "Production" as far as being deployed onto a server, but those sites are pretty much no traffic. I would definitely like to get some usage/overhead info and work on improving things for sure.

@randy-girard

This comment has been minimized.

Copy link
Owner

commented Sep 12, 2016

What do you think of this? @westonganger I had to update to support SQLite. It causes some issues with concurrency and what not, but this should get us rolling :)

@westonganger

This comment has been minimized.

Copy link
Author

commented Sep 12, 2016

Yeah this is a decent start to get the ball rolling.

randy-girard added a commit that referenced this issue Sep 13, 2016

randy-girard added a commit that referenced this issue Sep 13, 2016

Merge pull request #2 from randy-girard/simplify-setup
#1: Updated to support SQLite, and use a simpler setup.

@blasterpal blasterpal referenced this issue Apr 6, 2017

Open

Suggested Features to work on - WIP #6

4 of 13 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.