Skip to content

Added jerry-ext amalgamation#4992

Closed
dmazzella wants to merge 2 commits into
jerryscript-project:masterfrom
dmazzella:master
Closed

Added jerry-ext amalgamation#4992
dmazzella wants to merge 2 commits into
jerryscript-project:masterfrom
dmazzella:master

Conversation

@dmazzella

Copy link
Copy Markdown
Contributor

Added option to enable jerry-ext amalgamation

@LaszloLango LaszloLango added the enhancement An improvement label Nov 20, 2024
@LaszloLango LaszloLango added this to the Release 3.0.0 milestone Nov 20, 2024

@LaszloLango LaszloLango left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@robertsipka robertsipka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please rebase it with master, merge the commits into one and add the missing DCO to the end of the commit msg. Otherwise, it looks good to me.

gergocs pushed a commit to gergocs/jerryscript that referenced this pull request Nov 29, 2024
Added option to enable jerry-ext amalgamation

The implementation is based on PR jerryscript-project#4992. Only applied the requested changes
and rebase to master.

Co-authored-by: Damiano Mazzella damianomazzella@gmail.com
JerryScript-DCO-1.0-Signed-off-by: Gergo Csizi gergocs@inf.u-szeged.hu
gergocs added a commit to gergocs/jerryscript that referenced this pull request Nov 29, 2024
Added option to enable jerry-ext amalgamation

The implementation is based on PR jerryscript-project#4992. Only applied the requested changes
and rebase to master.

Co-authored-by: Damiano Mazzella damianomazzella@gmail.com
JerryScript-DCO-1.0-Signed-off-by: Gergo Csizi gergocs@inf.u-szeged.hu
@robertsipka

Copy link
Copy Markdown
Contributor

I'm closing it because the requests have been addressed here: #5184

LaszloLango pushed a commit that referenced this pull request Dec 3, 2024
Added option to enable jerry-ext amalgamation

The implementation is based on PR #4992. Only applied the requested changes
and rebase to master.

Co-authored-by: Damiano Mazzella damianomazzella@gmail.com
JerryScript-DCO-1.0-Signed-off-by: Gergo Csizi gergocs@inf.u-szeged.hu
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement An improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants