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

Support python3 & es5 #827

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Support python3 & es5 #827

wants to merge 9 commits into from

Conversation

bdh1011
Copy link

@bdh1011 bdh1011 commented Nov 24, 2016

I just found there was support_es5 branch #820 which was created after I forked.
This PR is about just for python3 and es5 support code. I really needed supporting es5 with python3 and finally I tried rough draft about it.
I hope someone would get some help from this. Please someone write some code for compatibility.

Note:

  • Only for es >= 5.0.0 & python >= 3.5 (tested)
  • Changed tox, requirements.txt, setup.py for just python 3.5 and es5

@bdh1011 bdh1011 mentioned this pull request Nov 25, 2016
@spotlesscoder
Copy link
Contributor

I think support for ES5 does already exist now. Is this PR still relevant?

@q2dg
Copy link

q2dg commented Jan 15, 2019

Please, close this PR. It confuses people as @CodingSpiderFox has asked

ajaywk7 pushed a commit to freshdesk/elastalert that referenced this pull request Feb 14, 2023
Helm Chart - Expose prometheus metrics to kubernetes prometheus-pod-sd
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.

3 participants