Skip to content
This repository was archived by the owner on Jan 27, 2026. It is now read-only.

fix(cli): cater for new potential error#2666

Merged
openshift-merge-bot[bot] merged 1 commit intojanus-idp:mainfrom
gashcrumb:RHIDP-6698
May 14, 2025
Merged

fix(cli): cater for new potential error#2666
openshift-merge-bot[bot] merged 1 commit intojanus-idp:mainfrom
gashcrumb:RHIDP-6698

Conversation

@gashcrumb
Copy link
Copy Markdown
Member

This change updates the existing condition where the CLI caters for when a package main script points to a .ts file. This addresses an update to the default behavior for how modules are handled by require.

This fixes RHIDP-6698

This change updates the existing condition where the CLI caters for when
a package main script points to a .ts file.  This addresses an update to
the default behavior for how modules are handled by `require`.

Signed-off-by: Stan Lewis <gashcrumb@gmail.com>
@gashcrumb gashcrumb requested review from a team, davidfestal, jesuino and kadel as code owners May 8, 2025 14:22
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented May 8, 2025

🦋 Changeset detected

Latest commit: 91008fe

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@janus-idp/cli Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@openshift-ci openshift-ci bot requested review from Fortune-Ndlovu and dzemanov May 8, 2025 14:22
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented May 8, 2025

@gashcrumb gashcrumb removed the request for review from jesuino May 8, 2025 14:26
@gashcrumb
Copy link
Copy Markdown
Member Author

/retest

@davidfestal
Copy link
Copy Markdown
Member

/lgtm

@openshift-merge-bot openshift-merge-bot bot merged commit 701b93c into janus-idp:main May 14, 2025
12 of 15 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants