Skip to content
This repository was archived by the owner on Jun 1, 2023. It is now read-only.

Fix operator implementation visibility#264

Merged
mattt merged 3 commits intoSwiftDocOrg:masterfrom
Ehlster:fix-operator-implementation-visibility
May 18, 2021
Merged

Fix operator implementation visibility#264
mattt merged 3 commits intoSwiftDocOrg:masterfrom
Ehlster:fix-operator-implementation-visibility

Conversation

@Ehlster
Copy link
Member

@Ehlster Ehlster commented Apr 26, 2021

The generated documentation could contain operator implementations which should not be included because of their access level.

@Ehlster Ehlster force-pushed the fix-operator-implementation-visibility branch from 767891b to 17d5cc9 Compare April 28, 2021 17:38
Ehlster added a commit to Ehlster/swift-doc that referenced this pull request May 11, 2021
@mattt
Copy link
Contributor

mattt commented May 18, 2021

Thanks, @Lukas-Stuehrk! Taking a look at #277 now.

@mattt mattt merged commit 4ec92a5 into SwiftDocOrg:master May 18, 2021
@Ehlster Ehlster deleted the fix-operator-implementation-visibility branch May 18, 2021 16:46
mattt added a commit that referenced this pull request Jun 1, 2021
* Fix the visibility of operator implementations.

* Only include operators if they have at least one visible implementation.

* Add changelog for #264.

* Bring back links to other symbols in declarations.

* Refactor the search of linked symbols.

* Add changelog entry for #277

Co-authored-by: Mattt <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants