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

Move RadioGroup from incubation to release #598

Merged
merged 1 commit into from
Jun 14, 2024

Conversation

mtrna
Copy link

@mtrna mtrna commented Jun 4, 2024

Hello and thank you for maintaining Nebula!

Would it be possible to add RadioGroup to release? A project I am maintaining is using it since ~2012.

Please let me know here in case there are any additional requirement regarding the transition.

linked issue: #515 (FormattedText to release)

@mtrna mtrna force-pushed the radiogroup-to-release branch from 83245c0 to eab6eb9 Compare June 4, 2024 07:28
Copy link
Contributor

@merks merks left a comment

Choose a reason for hiding this comment

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

A released version should have a version >= 1.0.0 so that needs attention too:

image

And if you change the version in the MANIFEST.MF you need to change the pom.xml accordingly.

@mtrna
Copy link
Author

mtrna commented Jun 4, 2024

commit 2: fixed the versions and organization within releng

@mtrna mtrna force-pushed the radiogroup-to-release branch from 08019dc to 59555ca Compare June 4, 2024 10:16
@merks
Copy link
Contributor

merks commented Jun 4, 2024

In general it's preferred to keep a PR to a single commit which you can do by amending and force pushing changes as described here:

eclipse-birt/birt#1701 (comment)

@merks
Copy link
Contributor

merks commented Jun 4, 2024

In any case, this looks improved... Let's see how the build goes...

@mtrna mtrna force-pushed the radiogroup-to-release branch from 59555ca to 434d55d Compare June 4, 2024 10:21
@mtrna
Copy link
Author

mtrna commented Jun 4, 2024

squashed & force pushed the commits

@merks
Copy link
Contributor

merks commented Jun 4, 2024

Thank you! ❤️

@merks merks requested a review from lcaron June 4, 2024 10:39
Copy link
Contributor

@merks merks left a comment

Choose a reason for hiding this comment

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

I have built this locally and it looks good to me.

image

@lcaron

It's not my decision to make what's promoted to non-incubating, but in terms of releng, the structure and changes look fine.

@lcaron
Copy link
Contributor

lcaron commented Jun 13, 2024

Hi

For me it's ok to move from Incubation to release. @wimjongman is it ok for you too ?

@wimjongman
Copy link
Contributor

Yes.

@merks
Copy link
Contributor

merks commented Jun 14, 2024

Since everyone agrees, I will merge.

@merks merks merged commit 3e01682 into EclipseNebula:master Jun 14, 2024
3 checks passed
@merks
Copy link
Contributor

merks commented Jun 14, 2024

This is in a latest nightly build:

https://download.eclipse.org/nebula/updates/nightly/latest

@mtrna mtrna deleted the radiogroup-to-release branch June 18, 2024 13:20
@mtrna
Copy link
Author

mtrna commented Jun 18, 2024

@merks @lcaron @wimjongman thank you! :)

@merks
Copy link
Contributor

merks commented Jun 18, 2024

It's open source great? ❤️

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.

4 participants