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

Fix for Decorating existing method #69

Merged
merged 2 commits into from
Nov 19, 2019
Merged

Fix for Decorating existing method #69

merged 2 commits into from
Nov 19, 2019

Conversation

arontsang
Copy link
Contributor

Added support for Decorator style replacement.

public static class StaticWithBaseAccess
{
    public static int ReplacedCount => 0;
}

[StaticReplacement(typeof(StaticWithBaseAccess))]
public static class StaticWithBaseAccessReplacement
{
    public static int ReplacedCount =>StaticWithBaseAccess.ReplacedCount + 1;
}

Bugs.
Does not support async await.
Perhaps does not support Lambdas.

Working to support other types of generator pattern types.

@SimonCropp
Copy link
Member

so given

Does not support async await.
Perhaps does not support Lambdas.

is this still a work in progress?

@arontsang
Copy link
Contributor Author

arontsang commented Aug 5, 2019 via email

@arontsang
Copy link
Contributor Author

Hey Simon, this PR is ready to merge.

@SimonCropp
Copy link
Member

Can u add some doco to the readme

@arontsang
Copy link
Contributor Author

arontsang commented Aug 10, 2019 via email

@arontsang
Copy link
Contributor Author

Can u add some doco to the readme

Done

@SimonCropp SimonCropp added this to the 2.2.0 milestone Nov 19, 2019
@SimonCropp SimonCropp merged commit f3b5bb9 into Fody:master Nov 19, 2019
@SimonCropp
Copy link
Member

@arontsang thanks for this. it is now deployed to nuget

@SimonCropp SimonCropp added the Bug label Jan 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants