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

[Builds] Remove elements unnecessary with the new version-bump workflow #2387

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

HannesWell
Copy link
Member

With the new reusable workflow for automated version increments in pull-requests, introduced with #2352, it is not necessary anymore to bump versions in advance during release preparation or the enable bumping if the property 'compare-version-with-baselines.skip' is set to false. Everything now happens automatically and these elements can therefore be removed.

Removing these elements avoids partly duplicated configuration and generally reduced the amount of configuration.

With the new reusable workflow for automated version increments in
pull-requests it is not necessary anymore to bump versions in advance
during release preparation or the enable bumping if the property
'compare-version-with-baselines.skip' is set to false.
Everything now happens automatically and these elements can therefore be
removed.
@HannesWell HannesWell force-pushed the remove-unnecessary-build-elements branch from c45aafc to e8d34a8 Compare September 26, 2024 19:22
@akurtakov
Copy link
Member

As thenew workflow is not yet setup on this repo, this one can not be pushed, right?

@HannesWell
Copy link
Member Author

As thenew workflow is not yet setup on this repo, this one can not be pushed, right?

This repository is not really the problem since the removed elements are (hardly) used here, but we should await until it's active in all other (active) SDK repositories. And while JDT should only be a matter of committer time, Equinox and SWT seem to need more adjustments and might take longer.

@@ -774,32 +774,6 @@
</repository>
</repositories>
</profile>
<profile>
<!-- This provile enables automatic version bumps when running the build -->
Copy link
Contributor

Choose a reason for hiding this comment

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

As mentioned last time this can be used locally as well, so I would prefer to keep the profile

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes it can be used. But I wonder if it is really used by a relevant number of people?
Personally I have never used it and didn't even know about before working in this area. Do you use it or know anybody that uses it?

Now that we have it automated at GH, I think this particular profile became even less relevant.

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.

3 participants