Skip to content

IntelliJ fixes - #100

Closed
MrSnapperVibes wants to merge 3 commits into
elastic:mainfrom
MrSnapperVibes:small-fixes
Closed

IntelliJ fixes#100
MrSnapperVibes wants to merge 3 commits into
elastic:mainfrom
MrSnapperVibes:small-fixes

Conversation

@MrSnapperVibes

Copy link
Copy Markdown
Contributor
  • Remove unused imports
  • Undefined name index

Hopefully I signed the contributor agreement correctly this time 😅

 - Remove unused imports
 - Undefined name `index`

@basepi basepi left a comment

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.

Looks great! Thank you!

@basepi

basepi commented Nov 28, 2023

Copy link
Copy Markdown
Contributor

@MrSnapperVibes Can you please close this PR and re-create it one more time? I think we hit something weird during the github outages over the last two days and I can't get those tests to run. Apologies for the inconvenience.

@MrSnapperVibes MrSnapperVibes mentioned this pull request Nov 28, 2023
@MrSnapperVibes
MrSnapperVibes deleted the small-fixes branch November 30, 2023 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants