Change gen.cs file location

[smellilac]

Previously, your .csproj file was configured to create a gen.cs file. at the intermediate level ( Debug/.net/... ). Therefore, Visual Studio (possible problem with other code editors) cannot include the gen.cs file in the assembly. Because of this, I did not have the Builtin class (CS0246). So I moved the gen.cs file to the project root folder.

[deepsource-io]

Here's the code health analysis summary for commits bf137bf..ed20f5c. View details on DeepSource ↗.

Analysis Summary

Analyzer	Status	Summary	Link
C#	✅ Success		View Check ↗

---

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

[smellilac]

Sorry about the previous ones pullrequests, sometimes GPG just kills 😁

[HEJOK254]

I'm thinking of changing the GeneratedFiles directory to just Generated, as I think it would sound better, and also it's already in the .gitignore

Not sure however if it shouldn't just be left as GeneratedFiles and added to the .gitignore, so it's not confused with the Generated directory that was already added to the .gitignore.

The actual gen.cs file shouldn't be tracked, since it's generated automatically on build, and not supposed to be edited manually.

[smellilac]

I'm thinking of changing the GeneratedFiles directory to just Generated, as I think it would sound better, and also it's already in the .gitignore

Not sure however if it shouldn't just be left as GeneratedFiles and added to the .gitignore, so it's not confused with the Generated directory that was already added to the .gitignore.

The actual gen.cs file shouldn't be tracked, since it's generated automatically on build, and not supposed to be edited manually.

It seems to me that given the current and potential size of the project, this is not so important, I mean that there will be no confusion with the names and the name of the folder reflects its essence.

As for the rest, yes, it’s really worth adding gen.cs to GeneratedFiles in gitignore.

If you agree, then I will do it

[HEJOK254]

It seems to me that given the current and potential size of the project, this is not so important, I mean that there will be no confusion with the names and the name of the folder reflects its essence.

As for the rest, yes, it’s really worth adding gen.cs to GeneratedFiles in gitignore.

If you agree, then I will do it

Good point
Besides, we can always change it later

You can change it, and I'll merge the PR.
Just remember to delete the gen.cs file. The Generated directory should already be in the .gitignore

[HEJOK254]

It seems like the build check is failing, which is probably just an issue with the workflow. I'll check if it builds correctly on my machine when I have time in about 2h and merge it.

Gonna have to fix the workflow aswell

[smellilac]

It seems like the build check is failing, which is probably just an issue with the workflow. I'll check if it builds correctly on my machine when I have time in about 2h and merge it.

Gonna have to fix the workflow aswell

The problem is that the compiler recognizes the Builtin class late. I tried to solve this, but did not find an adequate way to communicate the build time using code.

[HEJOK254]

After reading a bit, it turns out that everything that had to be done was just changing BeforeTargets in the Date build target. Now it works even with the old path, which uses $(IntermediateOutputPath), which I think is a better approach, since it doesn't clutter the project files. Please try it on your own machine and tell me if it works or not.

Sorry I'm finding out about this so late :/

I think if you could revert the commits in the PR and change BeforeTargets to something like CoreCompile, which seems to work properly, I could rename and merge the PR. Or you could close this one and open a new one, idk