Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Address the nil dereference #557

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.

Sign up for GitHub

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

Jump to bottom
Merged
aviator-app merged 1 commit into from
Apr 18, 2025
Merged

Address the nil dereference #557

aviator-app merged 1 commit into from
Apr 18, 2025

Conversation

@draftcode
Copy link
Contributor

@draftcode draftcode commented Apr 18, 2025 •
edited
Loading

No description provided.

@draftcode draftcode requested a review from a team as a code owner April 18, 2025 00:27
@aviator-app
Copy link
Contributor

aviator-app bot commented Apr 18, 2025 •
edited
Loading

Current Aviator status

Aviator will automatically update this comment as the status of the PR changes.
Comment /aviator refresh to force Aviator to re-examine your PR (or learn about other /aviator commands).

This PR was merged using Aviator.

See the real-time status of this PR on the Aviator webapp.
Use the Aviator Chrome Extension to see the status of your PR within GitHub.

@aviator-app
Copy link
Contributor

aviator-app bot commented Apr 18, 2025 •
edited
Loading

✅ FlexReview Status

Common Owner: aviator-co/engineering (expert-load-balance assignment)
Owner and Assignment:

  • ✅ aviator-co/engineering (expert-load-balance assignment)
    • Assigned
      tulioz (auto-assigned)
    Owned Files cmd/av/pr.go, cmd/av/sync.go

Review SLO: 7 business hours if PR size is <= 200 LOC for the first response.

@aviator-app aviator-app bot requested a review from tulioz April 18, 2025 00:27
@aviator-app
Copy link
Contributor

aviator-app bot commented Apr 18, 2025

This pull request failed to merge: some required checks failed. After you have resolved the problem, you should remove the blocked pull request label from this PR and then try to re-queue the PR. Note that the pull request will be automatically re-queued if it has the mergequeue label.

Failed checks: go-unit-test, golangci-lint

@aviator-app aviator-app bot added the blocked label Apr 18, 2025
@draftcode draftcode removed the blocked label Apr 18, 2025
@aviator-app aviator-app bot merged commit d6d9b3a into master Apr 18, 2025
5 of 7 checks passed
@aviator-app aviator-app bot deleted the fix_nil_deref branch April 18, 2025 01:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tulioz tulioz tulioz approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@draftcode @tulioz