Skip to content
This repository has been archived by the owner on May 14, 2021. It is now read-only.

Adds the Ruby rack application resource. #79

Closed
wants to merge 1 commit into from

Conversation

johnbellone
Copy link
Contributor

No description provided.

@coderanger
Copy link
Member

This seems oddly specific. What's the use case over the existing integrations?

@johnbellone
Copy link
Contributor Author

We're using the Grape API.

@coderanger
Copy link
Member

Okay, but you set this up as a service resource even though there is no service, it isn't particularly related to rack, and it is probably better served by a single ruby_execute resource?

@johnbellone
Copy link
Contributor Author

@coderanger Sorry, it looks like this may be better served as the dotenv being a new sub-resource. This deploy exactly like a Sinatra application.

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