Skip to content

Commit

Permalink
build(deps-dev): fix bump typescript-eslint from 8.4.0 to 8.6.0
Browse files Browse the repository at this point in the history
Handle 'no-duplicate-type-constituents' linting rule update introduced in 8.5.0.

Signed-off-by: Chris. Webster <[email protected]>
  • Loading branch information
webstech committed Sep 19, 2024
1 parent 25b2514 commit 129dd75
Show file tree
Hide file tree
Showing 5 changed files with 68 additions and 54 deletions.
2 changes: 1 addition & 1 deletion lib/commit-lint.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ export class LintCommit {
private messages: string[] = [];
private maxColumns = 76;

public constructor(patch: IPRCommit, options?: ILintOptions | undefined) {
public constructor(patch: IPRCommit, options?: ILintOptions) {
this.blocked = false;
this.lines = patch.message.split("\n");
this.patch = patch;
Expand Down
2 changes: 1 addition & 1 deletion lib/git.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ function trimTrailingNewline(str: string): string {
return str.replace(/\r?\n$/, "");
}

export function git(args: string[], options?: IGitOptions | undefined): Promise<string> {
export function git(args: string[], options?: IGitOptions): Promise<string> {
const workDir = options && options.workDir || ".";
if (options && options.trace) {
process.stderr.write(`Called 'git ${args.join(" ")}' in '${workDir}':\n${new Error().stack}\n`);
Expand Down
4 changes: 2 additions & 2 deletions lib/github-glue.ts
Original file line number Diff line number Diff line change
Expand Up @@ -192,7 +192,7 @@ export class GitHubGlue {
public async addPRCommitComment(pullRequest: pullRequestKeyInfo,
commit: string,
gitWorkDir: string | undefined,
comment: string, line?: number | undefined):
comment: string, line?: number):
Promise<{ id: number; url: string }> {
const prKey = getPullRequestKey(pullRequest);

Expand Down Expand Up @@ -248,7 +248,7 @@ export class GitHubGlue {
* @param {string} title the updated title
* @returns the PR number
*/
public async updatePR(prKey: pullRequestKey, body?: string | undefined, title?: string): Promise<number> {
public async updatePR(prKey: pullRequestKey, body?: string, title?: string): Promise<number> {
await this.ensureAuthenticated(prKey.owner);

const result = await this.client.rest.pulls.update({
Expand Down
112 changes: 63 additions & 49 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@
"ts-jest-resolver": "^2.0.1",
"ts-node": "^10.9.2",
"typescript": "^5.6.2",
"typescript-eslint": "8.4.0"
"typescript-eslint": "8.6.0"
},
"dependencies": {
"@octokit/auth-app": "^7.1.1",
Expand Down

0 comments on commit 129dd75

Please sign in to comment.