Skip to content

Conversation

@labkey-susanh
Copy link
Contributor

@labkey-susanh labkey-susanh commented Dec 8, 2025

Rationale

  • We no longer need to assure ourselves that we are using the intended version of XMLBeans
  • A more general task for purging NPM versions from Artifactory helps to keep things tidy
  • We can get the Git data we need for the module.xml file without the (dying) versioning plugin

Related Pull Requests

Changes

  • Remove logging of XMLBeans version
  • Add PurgeNpmVersions task
  • Remove use of versioning plugin in favor of some simple git commands

Copy link
Member

@labkey-tchad labkey-tchad left a comment

Choose a reason for hiding this comment

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

If this doesn't work on Windows, maybe we could use the JGit library used by our VCS license tool? https://github.com/LabKey/tools/blob/develop/vcslicense/src/org/labkey/vcslicense/VCSLicense.java

@labkey-susanh
Copy link
Contributor Author

If this doesn't work on Windows, maybe we could use the JGit library used by our VCS license tool? https://github.com/LabKey/tools/blob/develop/vcslicense/src/org/labkey/vcslicense/VCSLicense.java

Good thought to keep in mind. Fortunately, indications are that it does work on Windows as currently written.

### TBD
### 7.X.X
*Released*: TBD
(Earliest compatible LabKey version: 25.10)
Copy link
Contributor

Choose a reason for hiding this comment

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

25.12?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think this should still be compatible with 25.10 since the presence of the versioning plugin version shouldn't cause problems; we simply won't be using it.

@labkey-susanh labkey-susanh merged commit ddce975 into develop Dec 11, 2025
3 checks passed
@labkey-susanh labkey-susanh deleted the fb_miscCleaning branch December 11, 2025 14:51
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