Eileen M. Uchitelle
Eileen M. Uchitelle
I ran into this issue on my mac and came to some conclusions that may be able to help. I was debugging this application https://github.com/ScottRobbins/RailsMultipleDatabaseIssue) from this issue on Rails...
I know this was a long time ago, but I'm not sure what feature you're requesting. You want numbers with extensions to be parsed? Or you think that parsing them...
Thanks @trliner, I better understand the issue now. If anyone is interested in a PR that would be great. I'm not sure when I'll get around to implementing this, but...
Hi @locochris, thanks for the PR! :smile: Can you tell me what bug not discarding the whitespace is causing?
The test really isn't an example of how having whitespace would break since it existed prior to this PR. Can you update the commit message to explain that it fixes...
> The application is proprietary, so I can' share the repo publicly. No worries, we were asking you to make a new one that reproduces the issue you're seeing.
So we don't forget again in the future: The problem we hit with this PR is that we don't know which entry is the default and therefore should get `db/migrate`...
> If a non-primary database does not need migrations (and therefore theoretically need not have migrations_path config) Will `database_tasks: false` not work for this? That will ignore tasks for the...
Hey @masterkain is it possible this has been happening for some time but the error was different? The error changed in this PR https://github.com/rails/rails/pull/39723. I also found this issue https://github.com/rails/rails/issues/41606...
@uberllama yes it is a 5 year old issue. I had a fix for this issue that passes in Active Record but I broke the Active Model tests. Working towards...