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

Do not keep the module name in ModuleDef #153

Merged
merged 1 commit into from
Jan 13, 2025
Merged

Do not keep the module name in ModuleDef #153

merged 1 commit into from
Jan 13, 2025

Conversation

mkaput
Copy link
Member

@mkaput mkaput commented Jan 9, 2025

This field only used memory while it always has been easily retrievable
from the analysis database.


Stack:

⚠️ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

@mkaput mkaput requested a review from a team as a code owner January 9, 2025 15:38
@mkaput mkaput requested review from Draggu and integraledelebesgue and removed request for a team January 9, 2025 15:38
@mkaput mkaput changed the base branch from spr/main/6ce837dc to main January 10, 2025 13:45
@mkaput mkaput force-pushed the spr/main/fec3f114 branch from b616fd0 to d983859 Compare January 10, 2025 13:45
@mkaput mkaput mentioned this pull request Jan 10, 2025
@mkaput mkaput changed the base branch from main to spr/main/6ce837dc January 10, 2025 13:45
Base automatically changed from spr/main/6ce837dc to main January 13, 2025 08:06
This field only used memory while it always has been easily retrievable
from the analysis database.

commit-id:fec3f114
@mkaput mkaput force-pushed the spr/main/fec3f114 branch from d983859 to 29ed38b Compare January 13, 2025 09:20
@mkaput mkaput enabled auto-merge January 13, 2025 09:20
@mkaput mkaput added this pull request to the merge queue Jan 13, 2025
Merged via the queue into main with commit bb6550f Jan 13, 2025
11 checks passed
@mkaput mkaput deleted the spr/main/fec3f114 branch January 13, 2025 09:25
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.

3 participants