feat!: de-mavenize the build#182
Closed
joewiz wants to merge 2 commits into
Closed
Conversation
Replace Maven with a Node.js/Gulp build modeled on monex (eXist-db/monex#aa11953), and wire in semantic-release for automated releases. - Remove pom.xml, xar-assembly.xml, xquery-license-style.xml - Rewrite gulpfile.js to assemble the XAR end-to-end: stage src/main/xar-resources/ into .build/, compile sass, copy vendor assets, expand *.tmpl tokens via @existdb/gulp-replace-tmpl, and zip to dist/exist-function-documentation-<version>.xar - Restore expath-pkg.xml.tmpl and repo.xml.tmpl as the single source of truth for package metadata; move app fields (target, namespace, title, etc.) into package.json under "app" - Add .releaserc, commitlint.config.js, .husky/commit-msg, and scripts/update-repo-changelog.js for semantic-release - Replace Maven CI workflow with a Node build, install via @existdb/xst, Cypress test, and semantic-release job on master; add commitlint workflow - Add build.xml shim for downstream tooling that calls Ant - Drop maven ecosystem from dependabot.yml - Ignore .build/, dist/, .env, .claude/ BREAKING CHANGE: builds now require Node.js (lts/*) instead of Maven. Use \`npm ci && npm run build\` to produce the XAR.
This dep was carried over from an untracked template file written during an aborted prior attempt. Master's XQuery code doesn't use jinks-templates yet — declaring it here causes installs to 500 on page render. Restore parity with the Maven xar-assembly.xml deps (exist + html-templating only).
Member
Author
|
Closing in favor of a clean redo from master — this branch inadvertently picked up untracked-on-disk files from a prior aborted demavenize attempt (jinks-templates dep, speculative changelog entry). Replacement PR incoming. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
[This response was co-authored with Claude Code. -Joe]
Summary
Closes #180 by simplifying the release path. Modeled directly on monex@aa11953.
.build/→dist/exist-function-documentation-<version>.xar)expath-pkg.xml.tmpl/repo.xml.tmpl(restored from history) andpackage.json#app@existdb/xst→ cypress → release on masterbuild.xmlshim retained for downstream Ant callersBREAKING CHANGE: builds now require Node.js (
lts/*) instead of Maven.Test plan
npm ci && npm run buildproduces a valid XAR locallynpx semantic-release --dry-run(manually, before merge) picks up commits since v2.1.1 and proposes v2.2.0