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

update cloudflared pkg to 2025.1.0 #6390

Closed
wants to merge 1 commit into from

Conversation

luckman212
Copy link
Contributor

Description

Updates cloudflared package to 2025.1.0 version

Checklist

  • Build rule all-supported completed successfully
  • New installation of package completed successfully
  • Package upgrade completed successfully (Manually install the package again)
  • Package functionality was tested
  • Any needed documentation is updated/created

Type of change

  • Bug fix
  • New Package
  • Package update
  • Includes small framework changes
  • This change requires a documentation update (e.g. Wiki)

I only tested the arch-v1000-7.2 toolchain build because that's what I have (DS1621). I don't think there is any issue for other architectures. But others can test & confirm.

@luckman212 luckman212 mentioned this pull request Jan 13, 2025
10 tasks
cross/cloudflared/digests Outdated Show resolved Hide resolved
@luckman212
Copy link
Contributor Author

Ok @hgy59 I redid it with the line ending intact, and squashed

please confirm thx

@hgy59
Copy link
Contributor

hgy59 commented Jan 13, 2025

@luckman212 oops, didn't see this before: please create a dedicated branch for any PR.
Never use the master branch for a PR, otherwise you will not be able to keep your fork in sync with the upstream repository.

see section "Pull request" on https://github.com/SynoCommunity/spksrc/blob/master/CONTRIBUTING.md

@luckman212 luckman212 closed this by deleting the head repository Jan 13, 2025
@luckman212
Copy link
Contributor Author

@hgy59 got it. Lots of mistakes today. Ok... #6391

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