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.

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

PXP-6758 #138

Open
wants to merge 9 commits into
base: feat/textSearch
Choose a base branch
from
Open

PXP-6758 #138

wants to merge 9 commits into from

Conversation

scintillating7
Copy link

Jira Ticket: PXP-6758

New Features

  • Adds settings for creating ES indices, loaded from yaml files, that include the names and definitions of ES analyzers.
  • Adds analyzer names to mapping for keyword fields so that we can do text analysis.

Improvements

-rename settings.py -> config.py
-move some tests
-refactor find_paths to find_path

-rename settings.py -> config.py
-move some tests
-refactor find_paths to find_path
@lgtm-com
Copy link

lgtm-com bot commented Oct 26, 2020

This pull request introduces 2 alerts and fixes 1 when merging 825e00b into 7c9471b - view on LGTM.com

new alerts:

  • 1 for Unused import
  • 1 for Wrong number of arguments in a call

fixed alerts:

  • 1 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Oct 27, 2020

This pull request fixes 1 alert when merging a7ba105 into 7c9471b - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Oct 27, 2020

This pull request fixes 1 alert when merging 8973416 into 7c9471b - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Oct 27, 2020

This pull request fixes 1 alert when merging e6d72f1 into 7c9471b - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Oct 28, 2020

This pull request fixes 1 alert when merging 6ec79ec into 7c9471b - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Oct 28, 2020

This pull request fixes 1 alert when merging 40f3c93 into 7c9471b - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Oct 28, 2020

This pull request fixes 1 alert when merging d16b8a1 into 7c9471b - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Oct 30, 2020

This pull request fixes 1 alert when merging 3720488 into 7c9471b - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Oct 30, 2020

This pull request fixes 1 alert when merging 99d50d5 into 7c9471b - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Oct 30, 2020

This pull request fixes 1 alert when merging 5a08693 into 7c9471b - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Oct 30, 2020

This pull request fixes 1 alert when merging 9f431ad into 7c9471b - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Oct 30, 2020

This pull request fixes 1 alert when merging 383ce9b into 7c9471b - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Oct 30, 2020

This pull request fixes 1 alert when merging afcb703 into 7c9471b - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Oct 31, 2020

This pull request fixes 1 alert when merging cc1d671 into 7c9471b - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Nov 2, 2020

This pull request fixes 1 alert when merging a69ff05 into 7c9471b - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@scintillating7 scintillating7 marked this pull request as ready for review November 2, 2020 18:54
@lgtm-com
Copy link

lgtm-com bot commented Nov 2, 2020

This pull request fixes 1 alert when merging dbc3eee into 7c9471b - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@scintillating7 scintillating7 changed the base branch from master to feat/textSearch November 5, 2020 21:29
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.

1 participant