Skip to content

Conversation

@ds5678
Copy link
Contributor

@ds5678 ds5678 commented Oct 21, 2025

Note: the unit tests do not actually test the reason for this pull request. They were written while I was attempting to reproduce the issue, but none of the methods in the test games appear to exhibit the problem.

Resolves #492

@coveralls
Copy link

coveralls commented Oct 21, 2025

Pull Request Test Coverage Report for Build 18693904330

Details

  • 2 of 3 (66.67%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.05%) to 34.36%

Changes Missing Coverage Covered Lines Changed/Added Lines %
Cpp2IL.Core/Model/Contexts/MethodAnalysisContext.cs 1 2 50.0%
Totals Coverage Status
Change from base Build 18333364266: 0.05%
Covered Lines: 4196
Relevant Lines: 10849

💛 - Coveralls

@ds5678 ds5678 changed the title Interface methods should not have a base method Interfaces should not have a base type Oct 21, 2025
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.

Interface methods can have BaseMethod but shouldn't

2 participants