Skip to content

Remove cart v1#316

Merged
manu1080 merged 10 commits intomasterfrom
manu1080/doofinder-lib/clean_cart
Oct 3, 2025
Merged

Remove cart v1#316
manu1080 merged 10 commits intomasterfrom
manu1080/doofinder-lib/clean_cart

Conversation

@manu1080
Copy link

@manu1080 manu1080 commented Oct 2, 2025

Clean cart_v1
Update

@manu1080 manu1080 self-assigned this Oct 2, 2025
Copy link
Member

@sonic182 sonic182 left a comment

Choose a reason for hiding this comment

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

setup CI tests please, node 20 should work

Copy link
Member

@sonic182 sonic182 left a comment

Choose a reason for hiding this comment

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

setup CI tests please, node 20, 22 and 24 should work

Copy link

@davidmolinacano davidmolinacano left a comment

Choose a reason for hiding this comment

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

Change the version in the package.json (https://github.com/doofinder/js-doofinder/blob/master/package.json) and update the changelog

Copy link
Member

@sonic182 sonic182 left a comment

Choose a reason for hiding this comment

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

Update CHANGELOG please

Copy link

@davidmolinacano davidmolinacano left a comment

Choose a reason for hiding this comment

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

LGTM! 🏆 🎖️

@manu1080 manu1080 requested a review from sonic182 October 2, 2025 15:29
@manu1080 manu1080 merged commit 31a9bca into master Oct 3, 2025
4 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.

4 participants