Ensure /db is included in git repo of plugin #11692
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While generating a plugin that does not use the database, an empty
db/
folder will be created if theseeds.rb
file is removed from the folder. Git does not recognise empty directories, so it will not add the/db
folder to source. See the output here: https://github.com/schneems/default-rails-pluginThen any tests run on a remote machine will fail:
http://stackoverflow.com/questions/17955787/sqlite3cantopenexception-unable-to-open-database-file-while-testing-ruby-gem
Either we can include the
seeds.rb
file in the generate output ofrails plugin new
or we could add a blank.gitignore
file to ensure that the/db
folder is included in the git repo.I don't see a good reason for removing the file (it only has comments anyway) and all tests pass even when we keep the file in the repo.
You won't run into this problem is you run a migration in your plugin's tests since that creates a
db/migrate
folder.All tests pass Railties.