Skip to content

Conversation

@manu1080
Copy link

@manu1080 manu1080 commented Oct 2, 2025

  • Clean cart_v1
  • Update

@manu1080 manu1080 self-assigned this Oct 2, 2025
@manu1080 manu1080 requested a review from sonic182 October 2, 2025 13:39
Copy link
Contributor

@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.

composer.phar should not be here, this should be installed globally 🤔

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.

remove all doc/**/*.html, if it is auto-generated, do not commit it. if in a future we use github pages (which I doubt) a GH CI and another page will build it

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.

LGTM

@sonic182 sonic182 force-pushed the manu1080/doofinder-lib/clean_cart branch from 2113716 to 87f32a0 Compare October 2, 2025 15:00
Copy link
Contributor

@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 merged commit 4632344 into master Oct 3, 2025
51 of 64 checks passed
@sonic182
Copy link
Member

sonic182 commented Oct 3, 2025

Remember to upload new version to https://packagist.org/packages/doofinder/doofinder you may ask @JoeZ99 to know how

@JoeZ99
Copy link
Member

JoeZ99 commented Oct 3, 2025

Remember to upload new version to https://packagist.org/packages/doofinder/doofinder you may ask @JoeZ99 to know how

Ahem

@JoeZ99 here
I don't remember how to do it, sorry!

What I do remember is studying it, and figuring out how to do it in 30 minutes or so 😆

@sonic182 sonic182 deleted the manu1080/doofinder-lib/clean_cart branch October 6, 2025 07:39
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.

6 participants