Skip to content

Remove unused method argument#1740

Merged
charisk merged 1 commit intomainfrom
charisk/remove-unused-arg
Nov 10, 2022
Merged

Remove unused method argument#1740
charisk merged 1 commit intomainfrom
charisk/remove-unused-arg

Conversation

@charisk
Copy link
Copy Markdown
Contributor

@charisk charisk commented Nov 9, 2022

The argument doesn't seem to be used anywhere so I've deleted it.

Checklist

N/A:

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

@charisk charisk requested a review from a team as a code owner November 9, 2022 17:20
@charisk charisk merged commit 71297e3 into main Nov 10, 2022
@charisk charisk deleted the charisk/remove-unused-arg branch November 10, 2022 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aeisenberg aeisenberg aeisenberg approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@charisk @aeisenberg