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

Version 1.0.0 #7

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Version 1.0.0 #7

wants to merge 15 commits into from

Conversation

chochkov
Copy link

@chochkov chochkov commented Jan 23, 2017

This PR:

  • Removes the activerecord, activesupport, rspec dependencies
  • Deprecates support for automated regression testing built based on rspec extension tests.
  • Adds Travis CI integration (extension setup on Ubuntu is currently failing)
  • Drops rake dependency on the user side and moves the functionality into the bin/dumbo executable.
  • Fixes known bugs
  • Completes & Extends the existing testsuite
  • Updates Readme towards Version 0.0.1 functionality
  • improve Rspec matchers in the specs - e.g. remove the SQL.gsub(/^ {6}/, '')
  • Be aware of supported ruby/postgres versions and provide meaningful errors if used against unsupported version.
  • Introduce integration testing (e.g. test the bin/dumbo executable possibly on a Docker image)
  • Consider removing the requirement for config/database.yml. Instead have one ~/.dumborc config file with the DB credentials;

TODO:

  • test DB.transaction { }

@chochkov chochkov force-pushed the version-1.0.0 branch 3 times, most recently from 7246362 to a6f5b10 Compare January 30, 2017 16:03
@chochkov chochkov changed the title [WIP] Version 1.0.0 Version 1.0.0 Jan 31, 2017
@zeitgeist
Copy link

Consider adding support for automatic generation of upgrade/downgrade tests as mentioned in adjust/pg-country#1

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.

2 participants