[GitHub] metron issue #682: METRON-1081: Fix Alerts and Ops UI Notices file

2017-10-16 Thread cestella
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

2017-10-16 Thread ottobackwards
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

2017-10-16 Thread simonellistonball
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

2017-10-16 Thread ottobackwards
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

2017-10-16 Thread cestella
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

2017-10-06 Thread mmiklavc
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

2017-08-03 Thread ottobackwards
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

2017-08-03 Thread ottobackwards
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.
---