Oh yeah, I did that for conventional reasons...figured it would be OK to just have the option in any YAML file. But isn't pull_requests.yaml being used basically as a data store? If it's only being machine read then yeah I guess it's not necessary to have ERB in there.

On Mar 25, 2013, at 5:51 PM, Tom Van Eyck <notifications@github.com> wrote:

> In lib/helpers.rb:
> 
> > @@ -57,7 +60,9 @@ def PullRequestsData.get_path
> >  
> >    def PullRequestsData.read
> >      path = get_path
> > -    data = YAML.load(File.read(path)) if File.exists?(path)
> > +    raw_data = IO.read path
> > +    erbified_data = ERB.new(raw_data).result
> > +    data = YAML.load(erbified_data) if File.exists?(path)
> I'm all for ERBing the config file because of the Heroku use case, but I'm not sure I understand the benefit of ERBifying the pull_requests.yaml file.
> 
> —
> Reply to this email directly or view it on GitHub.