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

Fix/16+ check ci results #22

Merged
merged 5 commits into from
Aug 2, 2023
Merged

Fix/16+ check ci results #22

merged 5 commits into from
Aug 2, 2023

Conversation

duncdrum
Copy link
Contributor

@duncdrum duncdrum commented Jul 27, 2023

supersedes #17

Note: tests on 7.0.0-SNAPSHOT are failing

BREAKING CHANGES

  • bumps min exist to 5.5.1 in line with roaster
  • bumps min node to 18 (current its)

@duncdrum duncdrum force-pushed the fix/16+ branch 2 times, most recently from 4ec4ec7 to af5da9f Compare July 27, 2023 22:38
@@ -4,7 +4,7 @@
name="http://e-editiones.org/tuttle"
abbrev="tuttle" version="1.2.0" spec="1.0">
<title>Tuttle - Git for eXist-db</title>
<dependency processor="http://exist-db.org" semver-min="5.3.0"/>
<dependency processor="http://exist-db.org" semver-min="5.5.1"/>
Copy link
Member

Choose a reason for hiding this comment

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

Changing the minimum required processor version is a breaking change. Are you certain this is necessary?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Based on the principle that this is the earliest version I m willing to support yes. Does it run on earlier versions maybe…

Copy link
Contributor Author

Choose a reason for hiding this comment

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

also that's why it says BREAKING CHANGES in the op ;-)

@windauer windauer merged commit f9af03c into master Aug 2, 2023
4 of 6 checks passed
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.

failing tests due to GitHub authentication failure repository URL in package.json is incorrect
3 participants