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

Don't render the "$" for module #17302

Merged
merged 1 commit into from
Apr 24, 2023
Merged

Conversation

Dedelweiss
Copy link
Collaborator

@Dedelweiss Dedelweiss commented Apr 18, 2023

In this PR:

  • I deleted the $ in the file NameNormalizer

The ownerNameChain extension method generates a list of normalized owner names for a symbol, starting with the immediate owner and ending with the top-level owner.
The normalizedFullName extension method combines the results of ownerNameChain into a single string with "." separators.

Before:

Screenshot 2023-04-24 at 13 06 12

Screenshot 2023-04-24 at 13 06 21

### After:

Screenshot 2023-04-24 at 13 07 35

Screenshot 2023-04-24 at 13 07 42

Fixes: #15880

@Dedelweiss Dedelweiss marked this pull request as ready for review April 18, 2023 15:12
@Dedelweiss Dedelweiss force-pushed the weird_module branch 2 times, most recently from bdd1499 to 8263956 Compare April 20, 2023 11:20
Copy link
Member

@KacperFKorban KacperFKorban left a comment

Choose a reason for hiding this comment

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

It's a step in the right direction. Added a suggestion that should make it work in a more general case.
Also, could you squash your commits after applying the suggestions?

Change of how to correct the Issue
- Add a new method "normalizedFullName"
- Call the method "ClassLikeSupport"

Add the ownerNameChain function
- To erase the $ in all the hierarchy
Copy link
Member

@KacperFKorban KacperFKorban left a comment

Choose a reason for hiding this comment

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

LGTM

@KacperFKorban KacperFKorban merged commit 824295e into scala:main Apr 24, 2023
17 checks passed
@Kordyjan Kordyjan added this to the 3.3.1 milestone Aug 2, 2023
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.

Weird module class showing up in docs
4 participants