Skip to content

Fix dictionary usage related analyzer problems#1526

Merged
tonyqus merged 1 commit into
nissl-lab:masterfrom
lahma:fix-dictionary-usage
Mar 30, 2025
Merged

Fix dictionary usage related analyzer problems#1526
tonyqus merged 1 commit into
nissl-lab:masterfrom
lahma:fix-dictionary-usage

Conversation

@lahma
Copy link
Copy Markdown
Collaborator

@lahma lahma commented Mar 29, 2025

Removing suppressions and fixing problems prohibiting compilation. These are basically performance improvements.

@tonyqus tonyqus added the cicd label Mar 29, 2025
@tonyqus
Copy link
Copy Markdown
Member

tonyqus commented Mar 30, 2025

Can you help solve the conflict?

@tonyqus tonyqus added this to the NPOI 2.7.4 milestone Mar 30, 2025
@lahma lahma force-pushed the fix-dictionary-usage branch from 3af018e to 0b99218 Compare March 30, 2025 16:32
@lahma
Copy link
Copy Markdown
Collaborator Author

lahma commented Mar 30, 2025

Rebased and conflict resolved.

@tonyqus
Copy link
Copy Markdown
Member

tonyqus commented Mar 30, 2025

LGTM

@tonyqus tonyqus merged commit cc7c956 into nissl-lab:master Mar 30, 2025
@tonyqus tonyqus added performance and removed cicd labels Mar 30, 2025
@lahma lahma deleted the fix-dictionary-usage branch April 10, 2025 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants