Skip to content

#3941 add integration tests for TranspileMojo#3987

Merged
maxonfjvipon merged 3 commits into
objectionary:masterfrom
Suban05:3941-transpile-it
Mar 10, 2025
Merged

#3941 add integration tests for TranspileMojo#3987
maxonfjvipon merged 3 commits into
objectionary:masterfrom
Suban05:3941-transpile-it

Conversation

@Suban05

@Suban05 Suban05 commented Mar 10, 2025

Copy link
Copy Markdown
Contributor

Ref: #3941

@github-actions

github-actions Bot commented Mar 10, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

🚀 Performance Analysis

Test Base Score PR Score Change % Change Unit Mode
benchmarks.XmirBench.xmirToEO 174.543 177.142 2.599 1.49% ms/op Average Time
benchmarks.XmirBench.xmirToPhi 181.020 188.551 7.531 4.16% ms/op Average Time
benchmarks.XmirBench.xmirToSaltyPhi 150.059 151.030 0.971 0.65% ms/op Average Time

⚠️ Performance loss: benchmarks.XmirBench.xmirToEO is slower by 2.599 ms/op (1.49%)
⚠️ Performance loss: benchmarks.XmirBench.xmirToPhi is slower by 7.531 ms/op (4.16%)
⚠️ Performance loss: benchmarks.XmirBench.xmirToSaltyPhi is slower by 0.971 ms/op (0.65%)

@Suban05

Suban05 commented Mar 10, 2025

Copy link
Copy Markdown
Contributor Author

@maxonfjvipon please take a look

maxonfjvipon
maxonfjvipon suggested changes Mar 10, 2025
View reviewed changes
Comment thread eo-maven-plugin/src/test/java/org/eolang/maven/TranspileMojoIT.java Outdated
@maxonfjvipon maxonfjvipon added this pull request to the merge queue Mar 10, 2025
Merged via the queue into objectionary:master with commit 3d63cb2 Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@maxonfjvipon maxonfjvipon maxonfjvipon approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Suban05 @maxonfjvipon