Skip to content
This repository has been archived by the owner on Jan 11, 2024. It is now read-only.

Remove Interface static #11

Merged
merged 9 commits into from
Jan 9, 2024
Merged

Remove Interface static #11

merged 9 commits into from
Jan 9, 2024

Conversation

shargon
Copy link
Member

@shargon shargon commented Jan 6, 2024 •
edited

Preparing for dotnet standard 2.1

@coveralls
Copy link

coveralls commented Jan 6, 2024 •
edited

Pull Request Test Coverage Report for Build 7444191062

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.5%) to 81.001%
Totals Coverage Status
Change from base Build 7443931781: -0.5%
Covered Lines: 2045
Relevant Lines: 2460
💛 - Coveralls

This was referenced Jan 6, 2024
Closed
Closed
Closed
@shargon shargon requested a review from Jim8y January 6, 2024 15:07
@shargon
Copy link
Member Author

shargon commented Jan 8, 2024

merge?

@Jim8y
Copy link

Jim8y commented Jan 8, 2024

@cschuchardt88

@cschuchardt88
Copy link
Member

cschuchardt88 commented Jan 9, 2024 •
edited

@shargon @Jim8y looks good, but maybe check core applicationengine its used in contracts syscall.

@shargon shargon merged commit 144d8c7 into master Jan 9, 2024
1 check passed
@shargon shargon deleted the remove-interface-static branch January 9, 2024 06:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@cschuchardt88 cschuchardt88 cschuchardt88 requested changes

@Jim8y Jim8y Jim8y approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@shargon @coveralls @Jim8y @cschuchardt88