Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Adds a new parameter, max_analyzer_offset, for the highlighter #4035

Merged
merged 1 commit into from
Jul 28, 2022
Merged

Adds a new parameter, max_analyzer_offset, for the highlighter #4035

merged 1 commit into from
Jul 28, 2022

Conversation

hauck-jvsh
Copy link
Contributor

Change the version used to check if the new field is presented on the main branch.

@hauck-jvsh
Fix the version tests
67ad14d 
Signed-off-by: Hauck <joaoh14@gmail.com>
@hauck-jvsh hauck-jvsh requested review from a team and reta as code owners July 28, 2022 17:51
@hauck-jvsh hauck-jvsh changed the title Fix the version tests Adds a new parameter, max_analyzer_offset, for the highlighter Jul 28, 2022
@reta
Copy link
Collaborator

reta commented Jul 28, 2022

I think this one will fail and is going to be back to green after #4031

@hauck-jvsh
Copy link
Contributor Author

I think so too.

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@reta reta merged commit 85bfde1 into opensearch-project:main Jul 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@reta reta reta 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
@hauck-jvsh @reta