Skip to content

fix: no make target with project#689

Merged
RainyNight9 merged 2 commits intomainfrom
fix_ci
Jun 13, 2025
Merged

fix: no make target with project#689
RainyNight9 merged 2 commits intomainfrom
fix_ci

Conversation

@luohoufu
Copy link
Copy Markdown
Contributor

What does this PR do

This pull request updates the .github/workflows/release.yml file to modify how dependencies are added for non-Windows targets during the release workflow.

Changes to dependency management:

  • Replaced the make add-dep-pizza-engine-linux command with a direct cargo add command to add the pizza-engine dependency from a Git repository. This change also specifies additional features (query_string_parser and persistence) to be included when adding the dependency. ([.github/workflows/release.ymlL112-R112](https://github.com/infinilabs/coco-app/pull/689/files#diff-87db21a973eed4fef5f32b267aa60fcee5cbdf03c67fafdc2a9b553bb0b15f34L112-R112))

Rationale for this change

Standards checklist

  • The PR title is descriptive
  • The commit messages are semantic
  • Necessary tests are added
  • Updated the release notes
  • Necessary documents have been added if this is a new feature
  • Performance tests checked, no obvious performance degradation

@luohoufu luohoufu changed the title fix: no make with project fix: no make target with project Jun 13, 2025
@RainyNight9 RainyNight9 merged commit a8a9208 into main Jun 13, 2025
1 check passed
@RainyNight9 RainyNight9 deleted the fix_ci branch June 13, 2025 14:17
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.

3 participants