Skip to content

Conversation

@aglinxinyuan
Copy link
Contributor

@aglinxinyuan aglinxinyuan commented Dec 7, 2025

What changes were proposed in this PR?

Fix the IntelliJ config for macOS.

It seems like the ".main" string after each module name is not necessary for macOS.

If we include the ".main" string, IntelliJ may sometimes fail to find the module on macOS.

Screenshot 2025-12-06 at 21 45 19

Was this PR authored or co-authored using generative AI tooling?

No.

@aglinxinyuan aglinxinyuan self-assigned this Dec 7, 2025
@chenlica
Copy link
Contributor

chenlica commented Dec 7, 2025

@aglinxinyuan Can we make the PR subject more informative?

@aglinxinyuan aglinxinyuan changed the title fix: run configurations for MacOS fix: correct IntelliJ run configurations by removing unnecessary .main suffix Dec 8, 2025
@aglinxinyuan
Copy link
Contributor Author

@aglinxinyuan Can we make the PR subject more informative?

Updated.

Copy link
Contributor

@Xiao-zhen-Liu Xiao-zhen-Liu left a comment

Choose a reason for hiding this comment

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

LGTM.

@aglinxinyuan aglinxinyuan enabled auto-merge (squash) December 8, 2025 18:45
@aglinxinyuan aglinxinyuan merged commit 6b0a249 into main Dec 8, 2025
10 checks passed
@aglinxinyuan aglinxinyuan deleted the xinyuan-fix-macos-config branch December 8, 2025 18:52
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.

3 participants