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

ddns-scripts: Update ydns.io.json #26006

Merged
merged 1 commit into from
Feb 19, 2025
Merged

ddns-scripts: Update ydns.io.json #26006

merged 1 commit into from
Feb 19, 2025

Conversation

ACI0419
Copy link
Contributor

@ACI0419 ACI0419 commented Feb 18, 2025

Maintainer: @ACI0419

Description:
ddns-scripts: Update ydns.io.json to accept both 'good' and 'nochg' as valid responses for IPv4 and IPv6 updates.

@ACI0419 ACI0419 force-pushed the bugFix branch 3 times, most recently from fcaa5a6 to 9a7e0b0 Compare February 19, 2025 00:31
@ACI0419
Copy link
Contributor Author

ACI0419 commented Feb 19, 2025

I have modified the commit record, and now it should pass the CI/CD process.
: )

@ACI0419
Copy link
Contributor Author

ACI0419 commented Feb 19, 2025

Finally, I passed the CI/CD process!
But when I submitted two weeks ago, there didn't seem to be so many CI/CD processes. Have any new features been enabled?

@feckert
Copy link
Member

feckert commented Feb 19, 2025

Finally, I passed the CI/CD process! But when I submitted two weeks ago, there didn't seem to be so many CI/CD processes. Have any new features been enabled?

As far as I know, there have been no recent changes to the CI/CD. Also please bump the PKG_RELEASE by one

@ACI0419
Copy link
Contributor Author

ACI0419 commented Feb 19, 2025

Finally, I passed the CI/CD process! But when I submitted two weeks ago, there didn't seem to be so many CI/CD processes. Have any new features been enabled?

As far as I know, there have been no recent changes to the CI/CD. Also please bump the PKG_RELEASE by one

Done.And thank you for let me know this.

@feckert
Copy link
Member

feckert commented Feb 19, 2025

Can you please shorten the title of your commit? It's quite long and shouldn't be longer than 80 characters.
Thanks

@ACI0419
Copy link
Contributor Author

ACI0419 commented Feb 19, 2025

Can you please shorten the title of your commit? It's quite long and shouldn't be longer than 80 characters. Thanks

Done.

@feckert feckert merged commit 0a2746c into openwrt:master Feb 19, 2025
10 of 12 checks passed
@feckert
Copy link
Member

feckert commented Feb 19, 2025

Thanks merged!

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