Skip to content

TS for util.js#646

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

TS for util.js#646
1000TurquoisePogs wants to merge 6 commits intov3.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
Copy link
Copy Markdown

sonarqubecloud bot commented Feb 5, 2026

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