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

224: validate PolymorphicModels #225

Merged
merged 4 commits into from
May 30, 2024
Merged

224: validate PolymorphicModels #225

merged 4 commits into from
May 30, 2024

Conversation

LyricL-Gitster
Copy link
Contributor

Fixes #224

@LyricL-Gitster
Copy link
Contributor Author

i didn't take the time to install PG on my machine, so I couldn't run the tests locally. I do, however, have a similar instance set up in another project, so hopefully I was able to write the specs up correctly.

@jrochkind
Copy link
Owner

Lovely thank you!

I will merge and release today or tomorrow!

I appreciate it, thanks!

@LyricL-Gitster
Copy link
Contributor Author

OK, I actually installed PG to run the specs locally and figured out my bad reference in the new test. All's passing now! Thanks for the quick replies and all your effort on the gem!

@jrochkind
Copy link
Owner

ooh, you even added to the CHANGELOG, thank you! I or you should add credit to @LyricL-Gitster there -- I want to give you credit, plus it's good to show that people other than me are even occasionally contributing code!

@LyricL-Gitster
Copy link
Contributor Author

Sure thing! Should be good to go

@jrochkind
Copy link
Owner

oops, just remebered I forgot to come back to this after I left it with CI still running last week! Sorry! Merging and releasing today!

@jrochkind jrochkind merged commit 5fb2bd6 into jrochkind:master May 30, 2024
11 checks passed
@jrochkind
Copy link
Owner

2.3.1 released with this fix! Thank you very much @LyricL-Gitster

@LyricL-Gitster
Copy link
Contributor Author

Sure thing! Glad to help.

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.

PolymorphicModel validations not being run
2 participants