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

Allow a list of schemas when switching using schemas #154

Merged

Conversation

marksiemers
Copy link
Contributor

No description provided.

@marksiemers
Copy link
Contributor Author

@rpbaltazar - Apologies for the delay getting the other PR up-to-date. I started with a fresh fork of this repo and re-implemented the changes in the PR. See #112 for the discussion about the change.

@marksiemers
Copy link
Contributor Author

@rpbaltazar - Let me know if there is anything else I can help with here, further explanation, find another maintainer to do a code review.

Thanks!

Copy link
Contributor

@rpbaltazar rpbaltazar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for submitting. Sorry for my absense. I'm finding it very hard to keep up with work, family and OSS. I'll have this merged and it'll be included in the next release.

@rpbaltazar rpbaltazar merged commit b3937d6 into rails-on-services:development May 1, 2021
rpbaltazar added a commit that referenced this pull request Sep 13, 2021
**Implemented enhancements:**

- #112 - Allow a list of schemas when switching using schemas - #154
- Create schema between different versions of DB - #155 

**Fixed bugs:**

- #170 - Cache Key breaks on rails 6.0.4 - #171 

**Closed issues:**

- Add ruby 3 to the build matrix - #162 
- update rubocop and fixed broken rules - #157 
- added junit formatter and saving test output in circle - #172
@rpbaltazar rpbaltazar mentioned this pull request Feb 3, 2022
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