Skip to content

Removed 7.current references from travis CI matrix#242

Merged
alexcams merged 2 commits into
logstash-plugins:mainfrom
alexcams:remove-7-current-from-CI-matrix
Jan 22, 2026
Merged

Removed 7.current references from travis CI matrix#242
alexcams merged 2 commits into
logstash-plugins:mainfrom
alexcams:remove-7-current-from-CI-matrix

Conversation

@alexcams
Copy link
Copy Markdown
Contributor

@alexcams alexcams commented Jan 21, 2026

Necessary cleanup activity after end of support to 7.x LS version

@alexcams alexcams marked this pull request as draft January 21, 2026 15:07
@alexcams alexcams marked this pull request as ready for review January 21, 2026 15:17
Copy link
Copy Markdown
Contributor

@donoghuc donoghuc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I did see this

def extract_transport(client) # on 7.x client.transport is a ES::Transport::Client
but it seems out of scope and pretty harmless.

@donoghuc
Copy link
Copy Markdown
Contributor

Also, dont forget to PR 4.x as well :)

@alexcams alexcams merged commit ef4c5ad into logstash-plugins:main Jan 22, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants