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

Update us_state_courts.csv #314

Closed
wants to merge 2 commits into from

Conversation

ballotpedia
Copy link
Contributor

Add Washington Court of Appeals subdivisions

Add Washington Court of Appeals subdivisions
Copy link
Contributor

@jloutsenhizer jloutsenhizer left a comment

Choose a reason for hiding this comment

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

Looks good overall, please run compile.py to update the country-us.csv

@ballotpedia
Copy link
Contributor Author

Thanks for the comments @jloutsenhizer. Could we get some more guidance on the process for implementing these changes at some point? I am not totally clear on what sort of script you are asking us to run, or what kinds of approval is needed as we make these requests to the repo, so any summary of the steps involved would be helpful. Thanks! -Matt

@jloutsenhizer
Copy link
Contributor

Sorry for delays in replying.

The script I'm referring to is: https://github.com/opencivicdata/ocd-division-ids/blob/master/scripts/compile.py

It takes the files within in country directory and updates the country-xx.csv file. All PRs should use that script to ensure those files are up to date. There are also automated tests which verify this has been run. Example

As for approvals, the typical process is that at least 2 approvals are required. Typically once someone has reviewed the PR and approved it, we'll pull in another reviewer that's eligible to approve and merge the PR. Let me know if there's more questions.

@jloutsenhizer jloutsenhizer requested a review from jpmckinney July 27, 2022 17:15
@jpmckinney
Copy link
Member

Hmm, so we already have ocd-division/country:us/state:wa/court_of_appeals:1 for WA State Court of Appeals - 1st Division, etc.

I think the districts should hang off these existing parents, so ocd-division/country:us/state:wa/court_of_appeals:1/district:1, etc.

@jloutsenhizer
Copy link
Contributor

@jpmckinney I'm not sure if the districts of the divisions should be nested. Identifiers nesting is generally only used in connection with the level of jurisdiction of the governmental body reponsible for determining those districts. This was based on past discussions in:

Let me know if you think that rule doesn't apply here

@jpmckinney
Copy link
Member

If some software asks "How many court_of_appeals divisions are there in the State of Washington?", then after this PR, the answer would be 12. Right now, the answer is 3 (correct). I think the IDs need to be structured to answer this question correctly.

@jloutsenhizer
Copy link
Contributor

That's a fair point, I suppose either the types would have to be different on the district OCD IDs or nesting would also accomplish this. I think nesting looks cleaner so I agree with your recommendation

@ballotpedia
Copy link
Contributor Author

We've submitted a new pull request that addresses the requested changes #321

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants