Skip to content

Conversation

@ds5678
Copy link
Contributor

@ds5678 ds5678 commented May 15, 2025

Resolves #396

With BepInEx/Il2CppInterop#192 merged and https://github.com/BepInEx/Il2CppInterop/releases/tag/v1.5.0 released, we're now free to remove this field.

@coveralls
Copy link

coveralls commented May 15, 2025

Pull Request Test Coverage Report for Build 15052299381

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 1 (0.0%) changed or added relevant line in 1 file are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.005%) to 34.043%

Changes Missing Coverage Covered Lines Changed/Added Lines %
Cpp2IL.Core/ProcessingLayers/AttributeInjectorProcessingLayer.cs 0 1 0.0%
Files with Coverage Reduction New Missed Lines %
Cpp2IL.Core/ProcessingLayers/AttributeInjectorProcessingLayer.cs 2 0.64%
Totals Coverage Status
Change from base Build 15052278841: 0.005%
Covered Lines: 4155
Relevant Lines: 10794

💛 - Coveralls

@SamboyCoding SamboyCoding force-pushed the remove-attributeattribute-name branch from 9f6dfb1 to 8c7442b Compare May 15, 2025 18:18
@SamboyCoding SamboyCoding merged commit b3c0dbe into SamboyCoding:development May 15, 2025
5 checks passed
@ds5678 ds5678 deleted the remove-attributeattribute-name branch May 15, 2025 18:19
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.

Remove AttributeAttribute::Name

3 participants