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

Add miss and tenn seals #26

Merged
merged 5 commits into from
Jul 18, 2024
Merged

Add miss and tenn seals #26

merged 5 commits into from
Jul 18, 2024

Conversation

quevon24
Copy link
Member

@quevon24 quevon24 commented Jun 14, 2024

I added miss and tenn seals.

For the moment I added the mississippi judiciary seal while I wait for the response (I sent an email to the court asking for the seal) from the other seal I found:

image

@flooie
Copy link
Contributor

flooie commented Jun 17, 2024

I think this isnt quite right. The seal here is used by the workers comp boards but isnt the seal of the tenn. Supreme Court. think you need to add the workers comp courts to the list and link that Agriculture seal to them - not add it to the Supreme Court here

@quevon24
Copy link
Member Author

I already corrected the seals and, taking advantage of this, i added the correct one for Tennessee Supreme Court

@quevon24
Copy link
Member Author

quevon24 commented Jun 19, 2024

@flooie I think pre-commit.ci was configured for this repo but we never added the .pre-commit-config.yaml file, i added it and now precommit check works correctly

I also updated the checkout and setup-python actions in the workflow files

Right now I'm finding out why it modifies several svgs, I think it's because of the precommit

Update: Some svg and other files were updated because missing new line at the end of file (https://stackoverflow.com/questions/729692/why-should-text-files-end-with-a-newline)

@flooie flooie merged commit 13d12c7 into main Jul 18, 2024
6 checks passed
@flooie flooie deleted the add_seals branch July 18, 2024 22:32
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