ci(bun): migrating fission unit tests to bun [SYNTH-179] - #1355
Merged
Conversation
Dhruv-0-Arora
force-pushed
the
darora1/179/migrate-tests-bun
branch
from
June 26, 2026 21:23
6ce6905 to
51f4394
Compare
Dhruv-0-Arora
commented
Jun 30, 2026
|
Chorus detected one or more security issues with this pull request. See the Checks tab for more details. As a reminder, please follow the secure code review process as part of the Secure Coding Trust Commitment requirement. |
Collaborator
Author
|
I'm not sure where our consensus was on this PR. If we want to scrap it, then we should close this PR. Could be worth looking into though. I believe the reason we use npm was because in deployments we use yarn/npm. @PepperLola knows why |
PepperLola
changed the title
PepperLola
approved these changes
Jun 30, 2026
PepperLola
left a comment
PepperLola
left a comment
Member
There was a problem hiding this comment.
Since the tests pass and we now use bun to build/deploy, I'm fine with moving the unit tests over to bun as well.
Dhruv-0-Arora
commented
Jun 30, 2026
Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
Dhruv-0-Arora
force-pushed
the
darora1/179/migrate-tests-bun
branch
from
July 1, 2026 00:08
4d34ad5 to
7003bab
Compare
AlexD717
approved these changes
Jul 1, 2026
rutmanz
approved these changes
Jul 1, 2026
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Task
FissionUnitTest.yml is currently using npm and not bun.
SYNTH-179
Before merging, ensure the following criteria are met: