Skip to content

Conversation

@MikeMcQuaid
Copy link
Member

Check for Ruby files in taps that are outside of the detected Formula directory for a tap but inside one of the other potential directories.

This usually indicates a formula has been added in the wrong directory in a tap and is used to fail CI in this case.

Check for Ruby files in taps that are outside of the detected `Formula`
directory for a tap but inside one of the other potential directories.

This usually indicates a formula has been added in the wrong directory
in a tap and is used to fail CI in this case.
@MikeMcQuaid MikeMcQuaid added the in progress Maintainers are working on this label Dec 15, 2016
@MikeMcQuaid
Copy link
Member Author

@ilovezfs Any thoughts on this?

@ilovezfs
Copy link
Contributor

Won't this break

bash-4.4$ find homebrew-emacs|grep rb|grep -v Formula|grep -v '\.git'
homebrew-emacs/cmd/brew-emacs.rb
homebrew-emacs/Homebrew/emacs.rb
homebrew-emacs/Homebrew/emacs_formula.rb
bash-4.4$ find homebrew-php|grep rb|grep -v Formula|grep -v '\.git'
homebrew-php/Abstract/abstract-php-extension.rb
homebrew-php/Abstract/abstract-php-phar.rb
homebrew-php/Abstract/abstract-php-version.rb
homebrew-php/Abstract/abstract-php.rb
homebrew-php/language/php.rb
homebrew-php/Requirements/composer-requirement.rb
homebrew-php/Requirements/homebrew-php-requirement.rb
homebrew-php/Requirements/phar-building-requirement.rb
homebrew-php/Requirements/phar-requirement.rb
homebrew-php/Requirements/php-meta-requirement.rb

@MikeMcQuaid
Copy link
Member Author

@ilovezfs I will test, thanks. If it does: suggestions for a better test? Check for things with < Formula?

@MikeMcQuaid
Copy link
Member Author

Both work fine.

@MikeMcQuaid MikeMcQuaid merged commit aaf70eb into Homebrew:master Dec 25, 2016
@MikeMcQuaid MikeMcQuaid deleted the doctor-tap-repo-bad-ruby-files branch December 25, 2016 21:22
@MikeMcQuaid MikeMcQuaid removed the in progress Maintainers are working on this label Dec 25, 2016
@Homebrew Homebrew locked and limited conversation to collaborators May 3, 2018
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