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

Documentation #51

Open
starikcetin opened this issue Sep 21, 2021 · 5 comments
Open

Documentation #51

starikcetin opened this issue Sep 21, 2021 · 5 comments

Comments

@starikcetin
Copy link

Some documentation along with scenario examples of how this library comes handy would be a nice addition to the readme file.

@AraHaan
Copy link
Contributor

AraHaan commented Oct 4, 2021

If you want you can always add it to the readme if you are willing and have time to make a PR (feel free to open the PR).

@starikcetin
Copy link
Author

I do not have the time, unfortunately. Closing this then.

@AraHaan AraHaan reopened this Oct 4, 2021
@AraHaan
Copy link
Contributor

AraHaan commented Oct 4, 2021

I will see if I can document the usage then, so that way new users will be able to properly use the generator.

Also note: I plan to change the way I deploy the actual runtime code that is currently in nuget packages in the form of a special runtime pack that would need to eventually be installed as an .NET workload. But since dotnet workload install does not allow 3rd party workloads to be installed yet it will require a .NET global tool to be installed globally that would then handle all of the workload installation, uninstalation, and updating of the workload needed for it.

Also it would require a minor csproj change as well:

<UseElskomSdk>true</UseElskomSdk>

I will see about changing the default references to the other nuget packages the workload references as well (like analyzers). Also the cool thing about the workload thing is that the Sdk pack that will be in the workload will reference this generator automatically as well too.

@AraHaan
Copy link
Contributor

AraHaan commented Oct 30, 2021

I forgot to push 1.0.13 to nuget oops. Will push it seen and hopefully with documentation as well.

@AraHaan
Copy link
Contributor

AraHaan commented Oct 31, 2021

@starikcetin do you think this will work?
image

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

No branches or pull requests

2 participants