[Hawkular-dev] [Metrics] JAX-RS 1.1 implementation update

[Stefan Negrea]

----- Original Message -----
> From: "Thomas Segismont" <tsegismo at redhat.com>
> To: hawkular-dev at lists.jboss.org
> Sent: Friday, August 28, 2015 8:28:22 AM
> Subject: Re: [Hawkular-dev] [Metrics] JAX-RS 1.1 implementation update
> 
> I've been playing with GitHub API this morning, and I figured out to
> post the diff as a comment in pull requests.

Could the developer creating the PR add this diff manually? Or it has to be automatic on every PR? There are some PRs that do not touch the REST implementations. If we can selectively attach this to PRs that need it, I think we should do it even if it is verbose.

> 
> @metrics-dev would you find that useful? Or too noisy?

I would like to see an example first. 

> 
> I thought about this as I noted we don't always remind to make changes
> on both implementations.

We should require adding integration tests for any change in the REST interface. And since we started this dual implementation, we have been good at doing it. Adding this diff would be an excellent second check for the tedious configuration we currently have.

> 
> Le 13/08/2015 12:38, Thomas Segismont a écrit :
> > Forgot this: I believe this branch should be reviewed and merged ASAP.
> >
> > And, in the future, REST API PR reviewers should make sure changes/fixes
> > are applied to both implementations.
> >
> > Le 13/08/2015 12:32, Thomas Segismont a écrit :
> >> Hi,
> >>
> >> This morning I've fixed the remaining issues in the JAX-RS 1.1
> >> implementation branch. The REST API test suite is now fully passing
> >> against both JAX-RS 1.1 and 2.0 implementations.
> >>
> >> I've added a 'diff.txt' file in the 'api' [1]. Reviewers should
> >> carefully look at handler code differences.
> >>
> >> Regards,
> >> Thomas
> >>
> >>
> >> [1] diff -r --exclude=target metrics-api-jaxrs metrics-api-jaxrs-1.1 >
> >> diff.txt
> >> _______________________________________________
> >> hawkular-dev mailing list
> >> hawkular-dev at lists.jboss.org
> >> https://lists.jboss.org/mailman/listinfo/hawkular-dev
> >>
> >
> > _______________________________________________
> > hawkular-dev mailing list
> > hawkular-dev at lists.jboss.org
> > https://lists.jboss.org/mailman/listinfo/hawkular-dev
> >
> 
> _______________________________________________
> hawkular-dev mailing list
> hawkular-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/hawkular-dev
>