Bump versions

[krmahadevan]

Bumping versions

Closes #2972

[juherr]

Please, clean comments where you upgraded to the latest version

[krmahadevan]

@juherr - Please check now

[juherr]

I think it should be removed

[krmahadevan]

This cannot be removed because there are tests that are using this.

Please see https://github.com/testng-team/testng/blob/master/testng-core/src/test/java/test/reports/FailedReporterTest.java#L25

[juherr]

I'm talking about the commented version, not the dependency.

[juherr]

I think it should be removed

[krmahadevan]

This cannot be removed since it is being used here https://github.com/testng-team/testng/blob/master/testng-test-osgi/testng-test-osgi-build.gradle.kts#L21

[juherr]

I'm talking about the commented version, not the dependency.

[krmahadevan]

@juherr - They keep getting added back everytime refreshVersions gets executed.

The whole point of basically doing this exercise was because I am noticing that the publish snapshot is failing because there are uncommitted changes https://github.com/testng-team/testng/actions/runs/6941674340/job/18882998284#step:4:93

I am guessing that somehow publish is invoking refreshVersions

So can we please get this change merged so that I can figure out if that solves the problem. If not I would have to add -PallowUncommittedChanges as suggested in the error message.

[juherr]

Ok for this one. I supposed -b was for beta version that is before the release.

[krmahadevan]

Even for the other instance where in you had mentioned about removing them, i see the same behaviour. It looks like refreshVersions adds those comments back about new versions, the moment it detects them.