Skip to content

Conversation

@unidevel
Copy link
Contributor

Description

This PR

  1. Added the property release.autoPublish so that it can be controled by github action
  2. Removed the deprecated nexus-staging-maven-plugin

Motivation and Context

As part of the fix for PR #25879

Impact

Release 0.294

Test Plan

N/A

Contributor checklist

  • Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.

Release Notes

== NO RELEASE NOTE ==

@prestodb-ci prestodb-ci added the from:IBM PR from IBM label Aug 25, 2025
@unidevel unidevel marked this pull request as ready for review August 25, 2025 18:09
@unidevel unidevel requested a review from a team as a code owner August 25, 2025 18:09
@prestodb-ci prestodb-ci requested review from a team, bibith4 and infvg and removed request for a team August 25, 2025 18:09
@unidevel
Copy link
Contributor Author

I'll investigate and provide more info to central support team today, will try this tomorrow if no response.

@unidevel unidevel changed the title Add property release.autoPublish to pom.xml Add property release.autoPublish to pom.xml - DO NOT MERGE Aug 25, 2025
@unidevel unidevel changed the title Add property release.autoPublish to pom.xml - DO NOT MERGE Add property release.autoPublish to pom.xml Aug 26, 2025
@unidevel unidevel merged commit 3bd7d8c into prestodb:release-0.294 Aug 26, 2025
175 of 180 checks passed
unidevel added a commit that referenced this pull request Aug 26, 2025
## Description
Fix issue #25856, requires PR
#25880

## Motivation and Context

For this release, will push the artifacts for validation, then confirm
the publishing manually from [the central
portal](https://central.sonatype.com/), so that we can confirm all
artifacts validated before publishing.

In case of error,
1. Validation error - fix in the release branch and start a new workflow
to re-tag and publish
2. Partial published - fix in the release branch and start a new
workflow to re-tag and publish(can only publish the failed modules)

## Impact
Release 0.294

## Test Plan
N/A

## Contributor checklist

- [ ] Please make sure your submission complies with our [contributing
guide](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md),
in particular [code
style](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#code-style)
and [commit
standards](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#commit-standards).
- [ ] PR description addresses the issue accurately and concisely. If
the change is non-trivial, a GitHub Issue is referenced.
- [ ] Documented new properties (with its default value), SQL syntax,
functions, or other functionality.
- [ ] If release notes are required, they follow the [release notes
guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines).
- [ ] Adequate tests were added if applicable.
- [ ] CI passed.

## Release Notes

```
== NO RELEASE NOTE ==
```
unidevel added a commit to unidevel/presto that referenced this pull request Aug 28, 2025
This PR
1. Added the property `release.autoPublish` so that it can be controled
by github action
2. Removed the deprecated `nexus-staging-maven-plugin`

As part of the fix for PR prestodb#25879

Release 0.294

N/A

- [ ] Please make sure your submission complies with our [contributing
guide](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md),
in particular [code
style](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#code-style)
and [commit
standards](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#commit-standards).
- [ ] PR description addresses the issue accurately and concisely. If
the change is non-trivial, a GitHub Issue is referenced.
- [ ] Documented new properties (with its default value), SQL syntax,
functions, or other functionality.
- [ ] If release notes are required, they follow the [release notes
guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines).
- [ ] Adequate tests were added if applicable.
- [ ] CI passed.

```
== NO RELEASE NOTE ==
```
lga-zurich pushed a commit to lga-zurich/presto-exchange that referenced this pull request Sep 8, 2025
## Description
Fix issue prestodb#25856, requires PR
prestodb#25880

## Motivation and Context

For this release, will push the artifacts for validation, then confirm
the publishing manually from [the central
portal](https://central.sonatype.com/), so that we can confirm all
artifacts validated before publishing.

In case of error,
1. Validation error - fix in the release branch and start a new workflow
to re-tag and publish
2. Partial published - fix in the release branch and start a new
workflow to re-tag and publish(can only publish the failed modules)

## Impact
Release 0.294

## Test Plan
N/A

## Contributor checklist

- [ ] Please make sure your submission complies with our [contributing
guide](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md),
in particular [code
style](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#code-style)
and [commit
standards](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#commit-standards).
- [ ] PR description addresses the issue accurately and concisely. If
the change is non-trivial, a GitHub Issue is referenced.
- [ ] Documented new properties (with its default value), SQL syntax,
functions, or other functionality.
- [ ] If release notes are required, they follow the [release notes
guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines).
- [ ] Adequate tests were added if applicable.
- [ ] CI passed.

## Release Notes

```
== NO RELEASE NOTE ==
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

from:IBM PR from IBM

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants