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

refactor(ci): use cross #368

Merged
merged 10 commits into from
Apr 12, 2024
Merged

refactor(ci): use cross #368

merged 10 commits into from
Apr 12, 2024

Conversation

Itsusinn
Copy link
Member

@Itsusinn Itsusinn commented Apr 11, 2024

🤔 This is a ...

  • Refactoring
  • Workflow

🔗 Related issue link

#212

#210

@Itsusinn Itsusinn force-pushed the ci/use-cross branch 10 times, most recently from 768773d to 2696238 Compare April 11, 2024 15:34
clash_lib/Cargo.toml Outdated Show resolved Hide resolved
clash_lib/Cargo.toml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Show resolved Hide resolved
@ibigbug
Copy link
Member

ibigbug commented Apr 11, 2024

would be nice to manage the artifacts properly here, i.e. remove the whole huge "binaries" and get other individual build published
https://github.com/Watfaq/clash-rs/actions/runs/8650063287

image

@Itsusinn
Copy link
Member Author

Itsusinn commented Apr 11, 2024

https://github.com/Watfaq/clash-rs/actions/runs/8650063287
image

Strange. Why there are only two artifacts
You better check this normal one https://github.com/Watfaq/clash-rs/actions/runs/8649656339

remove the whole huge "binaries"

I will try delete it after release. Github will delete those artifacts in a few months, so i think it hardly affect anything

@Itsusinn
Copy link
Member Author

Itsusinn commented Apr 11, 2024

I will try delete it after release. Github will delete those artifacts in a few months, so i think it hardly affect anything

For now, binaries will only be kept for one day.

@Itsusinn Itsusinn force-pushed the ci/use-cross branch 2 times, most recently from ebf8003 to 93d079f Compare April 11, 2024 18:29
Copy link
Member

@ibigbug ibigbug left a comment

Choose a reason for hiding this comment

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

wonderful!

@ibigbug ibigbug enabled auto-merge (squash) April 12, 2024 13:13
@ibigbug ibigbug merged commit f04a96d into master Apr 12, 2024
19 checks passed
@ibigbug ibigbug deleted the ci/use-cross branch April 12, 2024 13:34
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.

2 participants