Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Update Connections to handle negative values#568

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

@aphralG
Copy link
Contributor

Proposed changes

Updated Connections fromunit64 toint64 to handle negative values being returned from NGINX Plus API.

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read theCONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

@aphralGaphralG requested a review froma team as acode ownerOctober 8, 2025 11:04
@nginx-bot
Copy link

Hi@aphralG!

Thanks for opening this pull request!
Be sure to check out ourContributing Guidelines while you wait for someone on the team to review this.

@nginx-botnginx-botbot added the communityIssues or PRs opened by an external contributor labelOct 8, 2025
@pdabelf5pdabelf5 merged commitbb2bf90 intonginx:mainOct 9, 2025
12 checks passed
@pdabelf5pdabelf5 added the bugAn issue reporting a potential bug labelOct 9, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@ciarams87ciarams87ciarams87 approved these changes

@vepatelvepatelvepatel approved these changes

@AlexFenlonAlexFenlonAlexFenlon approved these changes

@pdabelf5pdabelf5pdabelf5 approved these changes

Assignees

No one assigned

Labels

bugAn issue reporting a potential bugcommunityIssues or PRs opened by an external contributor

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@aphralG@ciarams87@vepatel@AlexFenlon@pdabelf5
[8]ページ先頭
©2009-2025 Movatter.jp