Skip to content

Conversation

@aicam
Copy link
Contributor

@aicam aicam commented Oct 8, 2025

Update

This PR fixes formatting issues that introduce redundant file changes in the core PR.

@Copilot Copilot AI review requested due to automatic review settings October 8, 2025 03:56
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR addresses code formatting issues in the access control service to prevent redundant file changes in the main core PR. The changes standardize code formatting by breaking long lines into multiple lines for better readability and removing extraneous blank lines.

  • Reformats long import statements and method calls to follow multi-line conventions
  • Removes unnecessary blank lines to maintain consistent spacing
  • Breaks long method signatures across multiple lines

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
AccessControlResourceSpec.scala Reformats import statements and long method calls to multi-line format
AccessControlService.scala Removes extra blank line and reformats method signature to multi-line format

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions github-actions bot added the backend Anything related to backend services label Oct 8, 2025
@aicam aicam requested review from bobbai00 and chenlica October 8, 2025 03:56
@aicam aicam self-assigned this Oct 8, 2025
Copy link
Contributor

@chenlica chenlica left a comment

Choose a reason for hiding this comment

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

LGTM

@aicam aicam merged commit e6c9d7d into apache:main Oct 8, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Anything related to backend services

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants