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

Support Rails 5.1 #299

Merged
merged 1 commit into from
Apr 28, 2017
Merged

Conversation

prathamesh-sonpatki
Copy link
Contributor

No description provided.

@prathamesh-sonpatki
Copy link
Contributor Author

Tested locally with a 5.1 app:

prathamesh ~/Projects/sources/codetriage 2.4.0 (codetriage/codetriage) -> rails-5-1
$ cat .env
GITHUB_APP_ID=78ac5b59c4798c7f2d23
GITHUB_APP_SECRET=858435d968d3b1042a1987964a6757baaead6efb
PORT=3000
GITHUB_API_KEY=af4f48bd58c3419234923a0800ec2a4171274816

prathamesh ~/Projects/sources/codetriage 2.4.0 (codetriage/codetriage) -> rails-5-1
$ rails c
EN[tunemygc] not enabled
VI, [2017-04-27T15:03:30.001265 #52384]  INFO -- : Loaded Configuration: /Users/prathamesh/Projects/sources/codetriage/config/scout_apm.yml. Using environment: development
Loading development environment (Rails 5.1.0)
>> ENV['GITHUB_APP_ID']
=> "78ac5b59c4798c7f2d23"
>> 

@rbhitchcock
Copy link

Works for my Rails 5.1 app as well 👍

@bkeepers bkeepers merged commit c22dac6 into bkeepers:master Apr 28, 2017
@bkeepers
Copy link
Owner

v2.2.1 is released

@prathamesh-sonpatki prathamesh-sonpatki deleted the support-rails-51 branch April 28, 2017 16:08
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.

4 participants