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.

Sign up for GitHub

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

Jump to bottom

Remove type column #67

Merged
merged 2 commits into from
Dec 14, 2020
Merged

Remove type column #67

merged 2 commits into from
Dec 14, 2020

Conversation

inglesp
Copy link
Contributor

@inglesp inglesp commented Dec 14, 2020

This removes the empty type column from the tables of parameters.

Before:

image

After:

image

wjchulme
wjchulme approved these changes Dec 14, 2020
View reviewed changes
Copy link
Contributor

@wjchulme wjchulme left a comment

Choose a reason for hiding this comment

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

Very glad I didn't spend more time trying to figure out how to do this. Thank you

@inglesp inglesp merged commit e7bcaff into master Dec 14, 2020
@inglesp inglesp deleted the remove-type-column branch December 14, 2020 13:36
StevenMaude added a commit that referenced this pull request Jul 20, 2021
@StevenMaude
Update the parameters.html override
8a80e4b 
To match the current version of mkdocstrings.

We previously introduced this override in #67 to remove the "Type"
column from the templated output. However, the underlying API has
changed. Therefore, this commit updates our override to match the
current version of the template.
StevenMaude added a commit that referenced this pull request Jul 20, 2021
@StevenMaude
Update the parameters.html override
76d7eca 
To match the current version of mkdocstrings.

We previously introduced this override in #67 to remove the "Type"
column from the templated output. However, the underlying API has
changed. Therefore, this commit updates our override to match the
current version of the template.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wjchulme wjchulme wjchulme approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@inglesp @wjchulme