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

Updated the JTS link to https #178

Merged
merged 1 commit into from Oct 6, 2020
Merged

Conversation

tomjenkinson
Copy link
Contributor

Signed-off-by: Tom Jenkinson tom.jenkinson@redhat.com

Signed-off-by: Tom Jenkinson <tom.jenkinson@redhat.com>
Copy link

@LanceAndersen LanceAndersen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and I validated the URL

@tomjenkinson
Copy link
Contributor Author

tomjenkinson commented Oct 5, 2020

Thanks @LanceAndersen - unfortunately you don't seem to have write access to this repo, I will raise a ticket with Eclipse as you should have (https://projects.eclipse.org/projects/ee4j.jta/who).

[edit: link to ticket https://bugs.eclipse.org/bugs/show_bug.cgi?id=567610]

I can't merge this (GitHub UI doesn't let me) so in the mean time if another project committer arrives here and would be able to review that would be great thanks.

Thanks again!

@tomjenkinson
Copy link
Contributor Author

@LanceAndersen the Eclipse Webmaster responded on https://bugs.eclipse.org/bugs/show_bug.cgi?id=567610 - it seems there is something that you would need to set on your Eclipse account. I hope if you are able to do that, then your existing review will release the GitHub "Rebase and merge" button for me.

@LanceAndersen
Copy link

LanceAndersen commented Oct 6, 2020 via email

@tomjenkinson
Copy link
Contributor Author

I checked mine and it does not have an "@" before tomjenkinson. I will update the ticket that you have it set.

@tomjenkinson
Copy link
Contributor Author

tomjenkinson commented Oct 6, 2020

Sorry, I see you have updated it - thanks!

@LanceAndersen
Copy link

LanceAndersen commented Oct 6, 2020 via email

@tomjenkinson
Copy link
Contributor Author

Thanks for the update Lance.

I would certainly expect they can fix it up but I am not sure how long it would take so in the meantime I will ask for a second review so that we can get this merged (GitHub won't allow me to merge).

@LanceAndersen
Copy link

LanceAndersen commented Oct 6, 2020 via email

@jgallimore jgallimore merged commit 222590c into jakartaee:master Oct 6, 2020
@tomjenkinson
Copy link
Contributor Author

@LanceAndersen - I think your reviews will allow GitHub merges now

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.

None yet

3 participants