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

Adapt to Jakarta Mail migration #1280

Merged
merged 3 commits into from
Jul 19, 2022
Merged

Adapt to Jakarta Mail migration #1280

merged 3 commits into from
Jul 19, 2022

Conversation

@basil basil added dependencies Pull requests that update a dependency file java Pull requests that update Java code labels Jul 19, 2022
@timja timja enabled auto-merge (squash) July 19, 2022 20:33
…execute (default) on project sample: Managed plugin dependency io.jenkins.plugins:jakarta-activation-api not listed in test classpath of sample plugin -> [Help 1]
…execute (default) on project sample: Managed plugin dependency io.jenkins.plugins:jakarta-mail-api not listed in test classpath of sample plugin -> [Help 1]
@jglick
Copy link
Member

jglick commented Jul 19, 2022

I am not exactly sure what you are force-pushing here—that breaks the incremental review function of PRs. It should not be necessary if the PR will be squash-merged anyway, right? (Except in weird cases like when you accidentally merged in the wrong base branch and filled the commit list with junk.)

@basil
Copy link
Member Author

basil commented Jul 19, 2022

I am not exactly sure what you are force-pushing here

You do not need to be

That breaks the incremental review function of PRs

Already aware of that

@timja timja merged commit 1b6da73 into jenkinsci:master Jul 19, 2022
@jetersen
Copy link
Member

Should we do a release of this before dropping for 2.289.x and 2.303.x?

@basil basil deleted the mail branch July 19, 2022 23:17
@basil
Copy link
Member Author

basil commented Jul 19, 2022

No strong preference, but if we do, then I'd like to get #1288 in first because that builds on top of this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file java Pull requests that update Java code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants