Fabien Ménager
Fabien Ménager
@jzabroski I think my proposal to rename the method should be applied before merging though. It will be much more clear imo
@jzabroski I hope you get better now, what do we do about the last remaining issues ? Do we merge this PR as is and I make a pull request...
Sorry, I meant the renaming of "AsExpression" to "Computed", but the todos are of course needed
I prepared the changes here, along with test fixes https://github.com/PhenX/fluentmigrator/tree/computed_columns
1. Computed, like Indexed, etc 2. OK, this feature will clearly need integration tests before beings released, but I think we will need another pull request.
@jzabroski let's merge this and after I'll make a PR with this what do you think? https://github.com/PhenX/fluentmigrator/tree/computed_columns_integration_tests
Hum, dumb question, but : did you approve it ?
@Shane32 no problem :)
Ok I'll update the dependabot file. About the config files some things may have changed since as it is somewhat possible via [Additional Files](https://github.com/dotnet/roslyn/blob/main/docs/analyzers/Using%20Additional%20Files.md) By the way Gerald Barré wrote...
> That guy is a prolific blogger and writes amazing .NET Documentation. I have too much internal code that has comments linking to his blog post, lol. But its often...