-
Notifications
You must be signed in to change notification settings - Fork 60
Problems migrating to 0.7 #193
Comments
So, let's start here: https://github.com/AArnott/CodeGeneration.Roslyn/wiki/Migrations This should be linked in the changelog. If you still have trouble, follow up. |
The link should probably be more prominently placed in the changelog though, now that I've looked at it ;D |
Thanks for the link. I get the same errors though. I have committed the changes to a new branch here in case you can see any obvious issues. (btw, did all the usual things, clean build, rebuild, delete obj/bin, restart VS, etc.) |
Hmm. Well, one obvious thing is that you set |
@amis92 Wow,that is bizarre! Not really sure, I must have picked it from somewhere, but can't see anything obvious, perhaps it was in the clipboard and I'd thought I'd copied the I replaced it the attribute value with the correct one. It seems there's been a breaking change in Roslyn, which causes a load of new errors (not related to the This: ParenthesizedLambdaExpression(body).WithParameters(ps); Throws a ParenthesizedLambdaExpression(ps, body); Once that was done, and I'd done a clean build, everything worked as expected. Thanks for your help 👍 I hope this gives a bit more confidence to the state of the |
Found it: CodeGeneration.Roslyn/samples/GeneratorInConsumerSolution/Sample.Consumer/Sample.Consumer.csproj Line 19 in dadd639
Must have copied the commented out section by mistake. |
Good catch. This'll get removed by #194 So, aside of Roslyn doing a breaking change, are you happy to approve the 0.7? :) |
@amis92 Yes, I think if it survives all my test code then the likelihood of a regression is very small. It includes a Record-type generators, Union-type generators, Reader monad generator, RWS monad generators, and several |
Closing as v0.7.57-alpha was released which should solve these problems: #191 (comment) |
In the interests of trying to help get the new version released quicker, I have updated my
LanguageExt.CodeGen.csproj
to use version0.7.5-alpha
to see if my test code works. I have pretty complex code-generators, so it should be a pretty good proving ground for any new features.It's not submitted to the public git, but the csproj for the
LanguageExt.CodeGen
project looks like this:The project builds OK.
I also have a scratchpad/testbed project that I use to quickly test various code-gen features. I changed the dependencies to
0.7.5-alpha
from0.6.1
. It would compile no problem with0.6.1
, but now I get the following:The project file for the testbed looks like this (again, only changed
0.6.1
to0.7.5-alpha
):It seems the code-gen tool is running, but none of the generators are found in the
LanguageExt.CodeGen
project.Any thoughts?
The text was updated successfully, but these errors were encountered: