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

docs: Updated docs to reference actually existing classes #219

Merged
merged 1 commit into from
Oct 24, 2023

Conversation

chriswk
Copy link
Member

@chriswk chriswk commented Oct 24, 2023

@sjaanus pointed out that we were referencing classes he couldn't find in the source code. After looking through, I see that we were indeed incorrectly naming the classes used. This PR updates the two references referred to.

@chriswk chriswk requested a review from sjaanus October 24, 2023 08:01
@chriswk chriswk self-assigned this Oct 24, 2023
@chriswk chriswk force-pushed the docs/updateClassReferences branch from 387daba to ec747ab Compare October 24, 2023 08:01
Copy link
Contributor

@sjaanus sjaanus left a comment

Choose a reason for hiding this comment

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

LGTM

@chriswk chriswk merged commit 4193824 into main Oct 24, 2023
4 checks passed
@chriswk chriswk deleted the docs/updateClassReferences branch October 24, 2023 08:23
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.

2 participants