Skip to content

TS for util.js - #646

Open
1000TurquoisePogs wants to merge 6 commits into
v3.x/stagingfrom
feat/v3/ts-utils
Open

TS for util.js#646
1000TurquoisePogs wants to merge 6 commits into
v3.x/stagingfrom
feat/v3/ts-utils

Conversation

@1000TurquoisePogs

Copy link
Copy Markdown
Member

This PR is a quick convert to typescript, to allow future typescript enhancements.
It is intended to be used alongside index.js - it will not work with index.ts because of the export statement differences.

Signed-off-by: 1000TurquoisePogs <sgrady@rocketsoftware.com>
Signed-off-by: 1000TurquoisePogs <sgrady@rocketsoftware.com>
Signed-off-by: 1000TurquoisePogs <sgrady@rocketsoftware.com>
@sonarqubecloud

Copy link
Copy Markdown

Signed-off-by: 1000TurquoisePogs <sgrady@rocketsoftware.com>
Signed-off-by: 1000TurquoisePogs <sgrady@rocketsoftware.com>
Signed-off-by: 1000TurquoisePogs <sgrady@rocketsoftware.com>
@struga0258
struga0258 self-requested a review February 5, 2026 02:30
@struga0258
struga0258 marked this pull request as ready for review February 5, 2026 02:30
@sonarqubecloud

sonarqubecloud Bot commented Feb 5, 2026

Copy link
Copy Markdown

Comment thread lib/util.ts
import * as BBPromise from 'bluebird';
import * as ipaddr from 'ipaddr.js';
import * as dns from 'node:dns';
import * as mergeUtils from '../utils/mergeUtils';

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.

mergeUtils not used.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Ready for Review

Development

Successfully merging this pull request may close these issues.

2 participants