David Goodlad dgoodlad

Organizations

@rubyaustralia @boxen
dgoodlad commented on issue boxen/puppet-java#58
@dgoodlad

@petems none of the previous maintainers have access to the boxen-downloads S3 bucket anymore, since it's owned by GitHub. @mikemcquaid can you pin…

dgoodlad commented on issue buildkite/agent#97
@dgoodlad

I could forget, true, but when setting up a build agent I'm far more aware of what its capabilities are. I'm in a better position to rationally eva…

dgoodlad commented on issue buildkite/agent#97
@dgoodlad

@keithpitt sure, but what if you forget? It's far safer to be able to whitelist acceptable jobs, rather than to accidentally run full builds which …

dgoodlad commented on issue buildkite/agent#97
@dgoodlad

:hand: we're in exactly this position. Switching to a white-list of acceptable jobs/metadata would be huge.

dgoodlad commented on pull request boxen/puppet-template#29
@dgoodlad

@julienlavergne sure, but GitHub's legal crew would need to :+1: it. Despite being a maintainer, neither I nor any of the others active folks here can…

dgoodlad commented on pull request boxen/puppet-template#29
@dgoodlad

@julienlavergne there are a few of us who still maintain Boxen in our free time, but there is no longer anyone full-time at GitHub doing so any more.

dgoodlad commented on pull request boxen/puppet-template#29
@dgoodlad

None of the previous maintainers of Boxen work at GitHub any more, so I don't feel that any of us have the authority to make such a change to the c…

dgoodlad commented on pull request boxen/our-boxen#721
@dgoodlad

@julienlavergne it may be regenerated if you've changed Puppetfile (by librarian-puppet, not by Puppet), but that shouldn't be happening when boots…

dgoodlad commented on pull request boxen/our-boxen#721
@dgoodlad

Without Puppetfile.lock, same as Gemfile.lock, you can end up with a dependency cycle – think authentication – when you bootstrap a brand new machine.

dgoodlad commented on pull request boxen/our-boxen#721
@dgoodlad

:-1: Puppetfile.lock is required, primarily for bootstrapping. Why do you want to remove it?

dgoodlad closed pull request boxen/our-boxen#721
@dgoodlad
added Puppetfile.lock into gitignore; Some cleaning is a good practice; Please merge it into master
@dgoodlad
mysql::grant can't find mysql on first run
dgoodlad commented on pull request boxen/puppet-homebrew#75
@dgoodlad

:+1:

dgoodlad commented on pull request boxen/puppet-java#33
@dgoodlad

I have no more objection to this; the reason I'd held off for quite awhile was worry about compatibility with ElasticSearch etc. I've spoken to a f…

dgoodlad commented on issue boxen/puppet-brewcask#36
@dgoodlad

This is an ordering issue; the boxen/puppet-git module tries to install its own git, but is doing so after brew cask has installed homebrew's defau…

dgoodlad commented on issue boxen/boxen#172
@dgoodlad

I think removing it makes more sense, personally. :+1:

dgoodlad commented on pull request boxen/puppet-ruby#128
@dgoodlad

You should be testing on system ruby, since Boxen only ever executes using system ruby. At the moment, this means ruby 2.0.0p481 (2014-05-08 revisi…

dgoodlad closed pull request boxen/puppet-ruby#12
@dgoodlad
Add rbenv_ruby native type
dgoodlad commented on pull request boxen/puppet-ruby#12
@dgoodlad

Looks like an old, now-unnecessary experiment.

@dgoodlad

:+1: looks legit to me!

dgoodlad commented on pull request boxen/puppet-homebrew#75
@dgoodlad

@mikemcquaid I really like the option of pushing those into the tap's cmd directory

@dgoodlad
CloudFront 2014-11-06 API removed
@dgoodlad

Thanks @hakunin; that path had obviously never been tested because Rails 4 was only in pre-release when it was written :grin: We're shipping 0.0.4 which…

@dgoodlad
dgoodlad deleted branch not-on-rails-4 at envato/rails_4_session_flash_backport
@dgoodlad
@dgoodlad
Enforce rails < 4.0
3 commits with 5 additions and 2 deletions
@dgoodlad
Rails 4 won't start
@dgoodlad
Enforce rails < 4.0
3 commits with 5 additions and 2 deletions