Skip to content

Commit

Permalink
chore: rename away more _ identifiers (#25481)
Browse files Browse the repository at this point in the history
`_` identifiers cause a problem with Java compilation. Remove them.

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
  • Loading branch information
rix0rrr authored May 8, 2023
1 parent d14c8d9 commit 1f4c4fa
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -257,7 +257,7 @@ function undefinedIfNoKeys<A extends { [key: string]: unknown }>(obj: A): A | un
* Explicitly configure no authorizers on specific HTTP API routes.
*/
export class HttpNoneAuthorizer implements IHttpRouteAuthorizer {
public bind(_: HttpRouteAuthorizerBindOptions): HttpRouteAuthorizerConfig {
public bind(_options: HttpRouteAuthorizerBindOptions): HttpRouteAuthorizerConfig {
return {
authorizationType: 'NONE',
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,7 @@ export interface IWebSocketRouteAuthorizer {
* Explicitly configure no authorizers on specific WebSocket API routes.
*/
export class WebSocketNoneAuthorizer implements IWebSocketRouteAuthorizer {
public bind(_: WebSocketRouteAuthorizerBindOptions): WebSocketRouteAuthorizerConfig {
public bind(_options: WebSocketRouteAuthorizerBindOptions): WebSocketRouteAuthorizerConfig {
return {
authorizationType: 'NONE',
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ export class HttpUrlIntegration extends HttpRouteIntegration {
super(id);
}

public bind(_: HttpRouteIntegrationBindOptions): HttpRouteIntegrationConfig {
public bind(_options: HttpRouteIntegrationBindOptions): HttpRouteIntegrationConfig {
return {
method: this.props.method ?? HttpMethod.ANY,
payloadFormatVersion: PayloadFormatVersion.VERSION_1_0, // 1.0 is required and is the only supported format
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ export class HttpLambdaIntegration extends HttpRouteIntegration {
});
}

public bind(_: HttpRouteIntegrationBindOptions): HttpRouteIntegrationConfig {
public bind(_options: HttpRouteIntegrationBindOptions): HttpRouteIntegrationConfig {
return {
type: HttpIntegrationType.AWS_PROXY,
uri: this.handler.functionArn,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ export class HttpServiceDiscoveryIntegration extends HttpPrivateIntegration {
super(id);
}

public bind(_: HttpRouteIntegrationBindOptions): HttpRouteIntegrationConfig {
public bind(_options: HttpRouteIntegrationBindOptions): HttpRouteIntegrationConfig {
if (!this.props.vpcLink) {
throw new Error('The vpcLink property is mandatory');
}
Expand Down

0 comments on commit 1f4c4fa

Please sign in to comment.