Skip to content

Conversation

@GeneralPoxter
Copy link
Contributor

No description provided.

@EntilZha
Copy link
Member

LGTM, aside from keeping the instructions for elastic search around.

@GeneralPoxter
Copy link
Contributor Author

GeneralPoxter commented Apr 26, 2021

All right, added that back in.

Edit: Should I include a note that elastic search is optional?

@EntilZha
Copy link
Member

Ya that would be helpful to add. Its mainly needed if you want to run the ES model, otherwise optional

@GeneralPoxter
Copy link
Contributor Author

All right done.

@EntilZha EntilZha merged commit a560c96 into Pinafore:master Apr 27, 2021
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