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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix validate_absence_of failing for array columns #1383

Conversation

thealiilman
Copy link
Contributor

@thealiilman thealiilman commented Dec 2, 2020

My very first time contributing to this gem. 馃 I love that the contributing guide is well-written! 馃帀

This fixes #1240.

Changelog

  • Made ValidatingAbsenceOfMatcher#value check whether column is array
  • Added tests on array columns for ValidateAbsenceOfMatcher

@thealiilman thealiilman force-pushed the fix/1240-validate-absence-of-array-column branch 2 times, most recently from 76d3e5f to 9140e2b Compare December 2, 2020 07:08
This commit fixes [thoughtbot#1240](thoughtbot#1240).

Changelog

- Made `ValidatingAbsenceOfMatcher#value` check whether column is array

- Added tests on array columns for `ValidateAbsenceOfMatcher`
@thealiilman thealiilman force-pushed the fix/1240-validate-absence-of-array-column branch from 9140e2b to 76b0ada Compare December 2, 2020 13:14
@thealiilman
Copy link
Contributor Author

Hmm, the jobs for the build on Travis CI seem to be stuck in Waiting to be queued state. Travis CI is operating though. 馃

Copy link
Collaborator

@mcmire mcmire left a comment

Choose a reason for hiding this comment

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

@thealiilman Very nice, looks perfect to me! In terms of CI, Travis just implemented some changes recently to open-source projects, so it's possible that we've exceeded the limits or something. I've restarted the build to see if that helped. Once we can get this to pass, this is good to go.

@thealiilman
Copy link
Contributor Author

@thealiilman Very nice, looks perfect to me! In terms of CI, Travis just implemented some changes recently to open-source projects, so it's possible that we've exceeded the limits or something. I've restarted the build to see if that helped. Once we can get this to pass, this is good to go.

Aha, I see. That may well be the cause.

Thanks for the approval!

@mcmire mcmire merged commit e19ebf2 into thoughtbot:master Dec 3, 2020
@mcmire
Copy link
Collaborator

mcmire commented Dec 3, 2020

Merged. Thank you so much!

@vsppedro
Copy link
Collaborator

vsppedro commented Dec 4, 2020

Thank you, @thealiilman!

@mcmire, we now have three bug fixes on the master. What do you think about a new release(v4.4.2) for them or do you have another idea in mind?

If you would like some help with that please let me know.

Bug fixes

EDIT: We have the 4.5.0 Milestone and it's only missing one PR. I will try to finish it. Would you like to add anything else?

@mcmire
Copy link
Collaborator

mcmire commented Dec 9, 2020

@vsppedro Oops, sorry, I thought I'd already replied to this. I will hold off on merging anything else, so what we currently have in the 4.5.0 milestone looks fine to me.

I am definitely not opposed to others handling releases, so if that's something you'd like to do, then that's totally cool. Let me know and I will make sure you are able to do this. Note that I have a PR around clarifying the instructions in MAINTAINING.md around releasing so hopefully it is relatively easy to follow them once that time comes: #1384

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.

validate_absence_of fails on Postgres array columns
3 participants