Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [jakartaee-tck-dev] Next steps for updating the jakartaee-tck repo...


On Fri, Apr 24, 2020 at 6:30 PM Cesar Hernandez <chernandez@xxxxxxxxxxxxx> wrote:
Thanks for the update,

I created the issue for jms [1] and also the PR is ready for review [2].
This time the PR is huge compared with the first one I made for ejb. I removed the jms signature changes since I understand now that those are generated.

Is there any process I need to perform to fully or partially test this and also others open PR's pending for review related with the javax->jakarta changes?

When reviewing pull requests, we have also looked for unneeded whitespace changes, such as adding/removing spaces at end of lines.  I learned this initially when making some javax => jakarta namespace changes for Jakarta EE SPEC APIs (I had my editor configured to remove trailing whitespace).  

Scott
 

On Fri, Apr 24, 2020 at 2:43 PM Scott Marlow <smarlow@xxxxxxxxxx> wrote:


As a side note, I notice this is not the first time this happens. Don't get me wrong I'm not angry at all. But I wonder if we can enable the https://github.com/eclipse-ee4j/jakartaee-tck/projects in order to keep track of who is doing what?

Good idea, I think we should try that.  We will need to decide between basic kanban vs automated kanban (there are a few others choices also).  

I started the https://github.com/eclipse-ee4j/jakartaee-tck/projects with the automated kanban template, nothing to much to see yet (need to add kanban cards as a next step). 

Scott
_______________________________________________
jakartaee-tck-dev mailing list
jakartaee-tck-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/jakartaee-tck-dev
_______________________________________________
jakartaee-tck-dev mailing list
jakartaee-tck-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/jakartaee-tck-dev

Back to the top