Skip to content

Comments

Attempt to fix publish.yml#606

Merged
brunoocasali merged 1 commit intomainfrom
brunoocasali-patch-1
Jan 9, 2025
Merged

Attempt to fix publish.yml#606
brunoocasali merged 1 commit intomainfrom
brunoocasali-patch-1

Conversation

@brunoocasali
Copy link
Member

@brunoocasali brunoocasali commented Jan 8, 2025

Pull Request

Although we don't need the ICU here, let's try this way... Otherwise we can try to disable it :)

Related issue

Fixes #605

What does this PR do?

  • ...

PR checklist

Please check if your PR fulfills the following requirements:

  • Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
  • Have you read the contributing guidelines?
  • Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

Copy link
Member

@curquiza curquiza left a comment

Choose a reason for hiding this comment

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

bors merge

@curquiza curquiza added the maintenance Anything related to maintenance (CI, tests, refactoring...) label Jan 9, 2025
@meili-bors
Copy link
Contributor

meili-bors bot commented Jan 9, 2025

Build succeeded:

@brunoocasali brunoocasali merged commit 9ed87aa into main Jan 9, 2025
3 checks passed
@brunoocasali brunoocasali deleted the brunoocasali-patch-1 branch January 9, 2025 14:46
@danFbach danFbach mentioned this pull request Jan 13, 2025
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Anything related to maintenance (CI, tests, refactoring...)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Impossible to release the package to Nugget

2 participants