[GitHub] metron issue #682: METRON-1081: Fix Alerts and Ops UI Notices file
Github user cestella commented on the issue: https://github.com/apache/metron/pull/682 Sorry otto, I didn't see that comment, I'll admit. I apologize. Let me take a look at them and if there's further notices work to do, I'll make a follow-on PR. Again, sorry, ignoring your discrepancy wasn't my intent. ---
[GitHub] metron issue #682: METRON-1081: Fix Alerts and Ops UI Notices file
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/682 I assume we are doing this for apache compliance. Our acceptance of this variance is compliant then. That was not my understanding. But I'm sure the chair knows. Thanks ---
[GitHub] metron issue #682: METRON-1081: Fix Alerts and Ops UI Notices file
Github user simonellistonball commented on the issue: https://github.com/apache/metron/pull/682 @ottobackwards I just re-ran this. We're not fixing versions in the UIs at the moment, so I would expect this kind of minor variance. This is mainly to ensure licenses are at least covered. The solution to the version fixing is the yarn stuff we're discussing elsewhere. ---
[GitHub] metron issue #682: METRON-1081: Fix Alerts and Ops UI Notices file
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/682 So, no comment on the discrepancy I found then? ---
[GitHub] metron issue #682: METRON-1081: Fix Alerts and Ops UI Notices file
Github user cestella commented on the issue: https://github.com/apache/metron/pull/682 +1 by inspection ---
[GitHub] metron issue #682: METRON-1081: Fix Alerts and Ops UI Notices file
Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/682 Is this still relevant? ---
[GitHub] metron issue #682: METRON-1081: Fix Alerts and Ops UI Notices file
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/682 Are the modules/versions pulled in dependent on the version of node or something? My list is different. I have more things, and different versions for some. [licenses.txt](https://github.com/apache/metron/files/1199144/licenses.txt) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] metron issue #682: METRON-1081: Fix Alerts and Ops UI Notices file
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/682 I am not sure how to verify or review this --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---