-
Notifications
You must be signed in to change notification settings - Fork 0
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
Document footprint links #229
Conversation
7317a0d
to
cf2ec2b
Compare
cc @fennisreed for feedback |
removed "County" from each county name as it was repetitive and may make parsing/reading through the table harder and it is already denoted by the column header
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This all looks efficient and great. Thanks for creating this so quickly, @ian-r-rose !
I believe the most recent commit broke the code snippets
…On Thu, Oct 26, 2023 at 3:48 PM britt ***@***.***> wrote:
***@***.**** approved this pull request.
This all looks efficient and great. Thanks for creating this so quickly,
@ian-r-rose <https://github.com/ian-r-rose> !
—
Reply to this email directly, view it on GitHub
<#229 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABLWQNYZ77ZZRBLQGAG3CUTYBLSCTAVCNFSM6AAAAAA6R3OCT6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTOMBQG42TANBUG4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
We both ran into the same failure on our commits, I am not sure how mine introduced anything new? @ian-r-rose |
This was the python issue in #231, it should work now |
Yes I saw that right after I commented, I misunderstood. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch, ty!
Fixes #222.
I used the following Python snippet to help generate the doc table:
Built docs here