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

change api #10

Draft
wants to merge 20 commits into
base: stable
Choose a base branch
from
Draft

change api #10

wants to merge 20 commits into from

Conversation

Gallimathias
Copy link
Member

No description provided.

* Add the verbatim value to the string type enum because this is the offical name in the c# documentation
* Add a ctor to ValueArgument that allows us to escape or non escape the Argument
* Add a TypeSyntax overload to the methodbuilder to allow to work with TypeSyntax objects from roslyn aswell
* Add diffrent overloads to declaration statement to allow to pass
  typesyntax diffrently and also working without any type
* Add the verbatim value to the string type enum because this is the offical name in the c# documentation
* Add a ctor to ValueArgument that allows us to escape or non escape the Argument
* Add a TypeSyntax overload to the methodbuilder to allow to work with TypeSyntax objects from roslyn aswell
* Add diffrent overloads to declaration statement to allow to pass
  typesyntax diffrently and also working without any type
* To support new features start to use the preview lang version
* Reorder enums to follow style guide
* Use correct using to prevent build issue. Is needed for obsolet flag
* Add new overloads to support also TypeSyntax and Parsing in Declaration statements
* Add a stringValueArgument to seperate String logic from Value logic and parsing
* New overloads are not optional because otherwise compiler can not decide between the diffrent overloads
* Use the property instead the smaller parameter because adding " happens only on Prop
* Fix tests so that all Expectation now correct
* There was a typo in the method names and this commit fix this typo. So assign is now spelled correctly
Resolve Conflicts:
	src/VaVare/Statements/DeclarationStatement.cs
@Gallimathias Gallimathias self-assigned this Jan 9, 2022
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

Successfully merging this pull request may close these issues.

1 participant