Skip to content

fix: rename __namedParameters to meaningful parameter names#29

Closed
TusharThakur04 wants to merge 1 commit intowebpack:mainfrom
TusharThakur04:fix/param-names
Closed

fix: rename __namedParameters to meaningful parameter names#29
TusharThakur04 wants to merge 1 commit intowebpack:mainfrom
TusharThakur04:fix/param-names

Conversation

@TusharThakur04
Copy link
Contributor

Summary
This PR renames default __namedParameters to meaningful param names (last word of type name)

Earlier:

deserialize(__namedParameters)

  • __namedParameters {ObjectDeserializerContext}
  • Returns: {void}

After:

deserialize(context)

  • context {ObjectDeserializerContext}
  • Returns: {void}

What kind of change does this PR introduce?
fix

Did you add tests for your changes?
no

Does this PR introduce a breaking change?
no

If relevant, what needs to be documented once your changes are merged or what have you already documented?

Use of AI
used ai for insight, discussion and faster code production.

@avivkeller
Copy link
Member

While I appreciate the intent here, we are trying to save the bulk of this work for GSoC. We got the fundamental workings down (basic pipeline), but a lot of the work here is for GSoC, which has not begun.

@avivkeller avivkeller closed this Mar 14, 2026
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.

2 participants