Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

replace @ignore with @hidden #12963

Merged
4 commits merged into from
Jan 5, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -161,8 +161,8 @@ export function makeConfigurationSettingEmpty(
}

/**
* @internal
* @hidden
* @internal
*/
export function transformKeyValue(kvp: KeyValue): ConfigurationSetting {
const obj: ConfigurationSetting & KeyValue = {
Expand All @@ -175,8 +175,8 @@ export function transformKeyValue(kvp: KeyValue): ConfigurationSetting {
}

/**
* @internal
* @hidden
* @internal
*/
export function transformKeyValueResponseWithStatusCode<
T extends KeyValue & HttpResponseField<any>
Expand All @@ -190,8 +190,8 @@ export function transformKeyValueResponseWithStatusCode<
}

/**
* @internal
* @hidden
* @internal
*/
export function transformKeyValueResponse<
T extends KeyValue & { eTag?: string } & HttpResponseField<any>
Expand Down
2 changes: 1 addition & 1 deletion sdk/appconfiguration/app-configuration/tsdoc.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@
"extends": [
"../../../tsdoc.json"
]
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,7 @@ export const ConnectionConfig = {

/**
* @internal
* @ignore
* @hidden
*/
export function isSharedAccessSignature(connectionString: string): boolean {
return connectionString.match(/;{0,1}SharedAccessSignature=SharedAccessSignature /) != null;
Expand Down
10 changes: 5 additions & 5 deletions sdk/core/core-amqp/src/errors.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import { isNode, isNumber, isString } from "../src/util/utils";
* Maps the conditions to the numeric AMQP Response status codes.
* @enum {ConditionStatusMapper}
* @internal
* @ignore
* @hidden
*/
export enum ConditionStatusMapper {
"com.microsoft:timeout" = AmqpResponseStatusCode.RequestTimeout,
Expand Down Expand Up @@ -457,7 +457,7 @@ export interface NetworkSystemError {

/**
* @internal
* @ignore
* @hidden
*/
const systemErrorFieldsToCopy: (keyof Omit<NetworkSystemError, "name" | "message">)[] = [
"address",
Expand Down Expand Up @@ -608,7 +608,7 @@ export function isSystemError(err: any): err is NetworkSystemError {

/**
* @internal
* @ignore
* @hidden
* Since browser doesn't differentiate between the various kinds of service communication errors,
* this utility is used to look at the error target to identify such category of errors.
* For more information refer to - https://html.spec.whatwg.org/multipage/comms.html#feedback-from-the-protocol
Expand All @@ -629,7 +629,7 @@ function isBrowserWebsocketError(err: any): boolean {

/**
* @internal
* @ignore
* @hidden
*/
const rheaPromiseErrors = [
// OperationTimeoutError occurs when the service fails to respond within a given timeframe.
Expand Down Expand Up @@ -729,7 +729,7 @@ export function translate(err: AmqpError | Error): MessagingError | Error {

/**
* @internal
* @ignore
* @hidden
*
* @param {*} error
* @returns {error is AmqpError}
Expand Down
2 changes: 1 addition & 1 deletion sdk/core/core-amqp/src/log.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ export const logger = createClientLogger("core-amqp");
/**
* Logs the error's stack trace to "verbose" if a stack trace is available.
* @param error Error containing a stack trace.
* @ignore
* @hidden
*/
export function logErrorStackTrace(error: any): void {
if (error && error.stack) {
Expand Down
8 changes: 4 additions & 4 deletions sdk/core/core-amqp/src/requestResponseLink.ts
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ export interface SendRequestOptions {

/**
* @internal
* @ignore
* @hidden
*/
export interface DeferredPromiseWithCallback {
resolve: (value?: any) => void;
Expand Down Expand Up @@ -230,7 +230,7 @@ export class RequestResponseLink implements ReqResLink {
}
/**
* @internal
* @ignore
* @hidden
*
* Type used in getCodeDescriptionAndError to get the normalized info from the responses emitted by EventHubs and ServiceBus.
*/
Expand All @@ -242,7 +242,7 @@ type NormalizedInfo = {

/**
* @internal
* @ignore
* @hidden
*
* Handle different variations of property names in responses emitted by EventHubs and ServiceBus.
*
Expand Down Expand Up @@ -271,7 +271,7 @@ export const getCodeDescriptionAndError = (props: any): NormalizedInfo => {
* 5. User's code after the sendRequest continues.
*
* @internal
* @ignore
* @hidden
*/
export function onMessageReceived(
context: Pick<EventContext, "message">,
Expand Down
4 changes: 2 additions & 2 deletions sdk/core/core-amqp/src/retry.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import { checkNetworkConnection } from "./util/checkNetworkConnection";

/**
* Determines whether the object is a Delivery object.
* @ignore
* @hidden
*/
function isDelivery(obj: any): boolean {
let result: boolean = false;
Expand Down Expand Up @@ -119,7 +119,7 @@ export interface RetryConfig<T> {

/**
* Validates the retry config.
* @ignore
* @hidden
*/
function validateRetryConfig<T>(config: RetryConfig<T>): void {
if (!config.operation) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

/**
* Checks whether a network connection is detected.
* @ignore
* @hidden
* @internal
*/
export function checkNetworkConnection(): Promise<boolean> {
Expand Down
2 changes: 1 addition & 1 deletion sdk/core/core-amqp/src/util/checkNetworkConnection.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import { logger } from "../log";

/**
* Checks whether a network connection is detected.
* @ignore
* @hidden
* @internal
*/
export function checkNetworkConnection(host: string): Promise<boolean> {
Expand Down
12 changes: 6 additions & 6 deletions sdk/core/core-amqp/src/util/runtimeInfo.browser.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,21 +3,21 @@

/**
* @internal
* @ignore
* @hidden
*
* @interface Window
*/
interface Window {}

/**
* @internal
* @ignore
* @hidden
*/
declare let self: Window & typeof globalThis & { navigator: Navigator };

/**
* @internal
* @ignore
* @hidden
*
* @interface Navigator
*/
Expand All @@ -30,7 +30,7 @@ interface Navigator {

/**
* Returns information about the platform this function is being run on.
* @ignore
* @hidden
* @internal
*/
export function getPlatformInfo(): string {
Expand All @@ -39,7 +39,7 @@ export function getPlatformInfo(): string {

/**
* Returns information about Node.js this function is being run on.
* @ignore
* @hidden
* @internal
*/
export function getFrameworkInfo(): string {
Expand All @@ -48,7 +48,7 @@ export function getFrameworkInfo(): string {

/**
* @internal
* @ignore
* @hidden
*
* @returns {string}
*/
Expand Down
4 changes: 2 additions & 2 deletions sdk/core/core-amqp/src/util/runtimeInfo.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import * as os from "os";

/**
* Returns information about the platform this function is being run on.
* @ignore
* @hidden
* @internal
*/
export function getPlatformInfo(): string {
Expand All @@ -14,7 +14,7 @@ export function getPlatformInfo(): string {

/**
* Returns information about Node.js this function is being run on.
* @ignore
* @hidden
* @internal
*/
export function getFrameworkInfo(): string {
Expand Down
20 changes: 10 additions & 10 deletions sdk/core/core-amqp/src/util/utils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import { WebSocketImpl } from "rhea-promise";
export { AsyncLock };
/**
* @internal
* @ignore
* @hidden
*
* Describes the options that can be provided to create an async lock.
*/
Expand Down Expand Up @@ -56,7 +56,7 @@ export interface WebSocketOptions {

/**
* @internal
* @ignore
* @hidden
*
* A constant that indicates whether the environment is node.js or browser based.
*/
Expand Down Expand Up @@ -117,7 +117,7 @@ export function parseConnectionString<T>(connectionString: string): ParsedOutput

/**
* @internal
* @ignore
* @hidden
*
* Gets a new instance of the async lock with desired settings.
* @param {AsyncLockOptions} [options] The async lock options.
Expand All @@ -134,7 +134,7 @@ export const defaultLock: AsyncLock = new AsyncLock({ maxPending: 10000 });

/**
* @internal
* @ignore
* @hidden
*
* Describes a Timeout class that can wait for the specified amount of time and then resolve/reject
* the promise with the given value.
Expand Down Expand Up @@ -236,7 +236,7 @@ export function delay<T>(

/**
* @internal
* @ignore
* @hidden
*
* Generates a random number between the given interval
* @param {number} min Min number of the range (inclusive).
Expand All @@ -248,7 +248,7 @@ export function randomNumberFromInterval(min: number, max: number): number {

/**
* @internal
* @ignore
* @hidden
*
* Type declaration for a Function type where T is the input to the function and V is the output
* of the function.
Expand All @@ -257,7 +257,7 @@ export type Func<T, V> = (a: T) => V;

/**
* @internal
* @ignore
* @hidden
*
* Executes an array of promises sequentially. Inspiration of this method is here:
* https://pouchdb.com/2015/05/18/we-have-a-problem-with-promises.html. An awesome blog on promises!
Expand All @@ -282,7 +282,7 @@ export function executePromisesSequentially(

/**
* @internal
* @ignore
* @hidden
*
* Determines whether the given connection string is an iothub connection string.
* @param {string} connectionString The connection string.
Expand All @@ -300,15 +300,15 @@ export function isIotHubConnectionString(connectionString: string): boolean {
}

/**
* @ignore
* @hidden
* @internal
*/
export function isString(s: any): s is string {
return typeof s === "string";
}

/**
* @ignore
* @hidden
* @internal
*/
export function isNumber(n: any): n is number {
Expand Down
4 changes: 4 additions & 0 deletions sdk/core/core-amqp/tsdoc.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
{
"$schema": "https://developer.microsoft.com/json-schemas/tsdoc/v0/tsdoc.schema.json",
"extends": ["../../../tsdoc.json"]
}
6 changes: 3 additions & 3 deletions sdk/core/core-client/src/base64.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
/**
* Encodes a string in base64 format.
* @param value the string to encode
* @internal @ignore
* @internal @hidden
*/
export function encodeString(value: string): string {
return Buffer.from(value).toString("base64");
Expand All @@ -13,7 +13,7 @@ export function encodeString(value: string): string {
/**
* Encodes a byte array in base64 format.
* @param value the Uint8Aray to encode
* @internal @ignore
* @internal @hidden
*/
export function encodeByteArray(value: Uint8Array): string {
// Buffer.from accepts <ArrayBuffer> | <SharedArrayBuffer>-- the TypeScript definition is off here
Expand All @@ -25,7 +25,7 @@ export function encodeByteArray(value: Uint8Array): string {
/**
* Decodes a base64 string into a byte array.
* @param value the base64 string to decode
* @internal @ignore
* @internal @hidden
*/
export function decodeString(value: string): Uint8Array {
return Buffer.from(value, "base64");
Expand Down
2 changes: 1 addition & 1 deletion sdk/core/core-client/src/createSpan.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ type OperationTracingOptions = OperationOptions["tracingOptions"];

/**
* Creates a function called createSpan to create spans using the global tracer.
* @ignore
* @hidden
* @param spanConfig The name of the operation being performed.
* @param tracingOptions The options for the underlying http request.
*/
Expand Down
4 changes: 2 additions & 2 deletions sdk/core/core-client/src/interfaceHelpers.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import { MapperTypeNames } from "./serializer";
import { OperationSpec, OperationParameter } from "./interfaces";

/**
* @internal @ignore
* @internal @hidden
*/
export function isStreamOperation(operationSpec: OperationSpec): boolean {
for (const statusCode in operationSpec.responses) {
Expand All @@ -24,7 +24,7 @@ export function isStreamOperation(operationSpec: OperationSpec): boolean {
* Get the path to this parameter's value as a dotted string (a.b.c).
* @param parameter The parameter to get the path string for.
* @returns The path to this parameter's value as a dotted string.
* @internal @ignore
* @internal @hidden
*/
export function getPathStringFromParameter(parameter: OperationParameter): string {
const { parameterPath, mapper } = parameter;
Expand Down
2 changes: 1 addition & 1 deletion sdk/core/core-client/src/operationHelpers.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ import {
} from "./interfaces";

/**
* @internal @ignore
* @internal @hidden
* Retrieves the value to use for a given operation argument
* @param operationArguments The arguments passed from the generated client
* @param parameter The parameter description
Expand Down
Loading