-
Notifications
You must be signed in to change notification settings - Fork 2
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
Format-dates #31
Format-dates #31
Conversation
bpepple
commented
Sep 11, 2021
- Fix date formatting for creator date of death
- Use Meta class to handle primary date formats
While we're updating the dates I think it'd also be good to change the DoB and DoD to be |
Also the tests should pass if you include the Github Secret |
Thought I'd set it, but I'll check it again on my fork. |
Not a bad idea. Give me a minute and I'll push a new commit to change that. |
Looks like my GH Secret Key is still not working. Really, I think an expiring testing cache/env secrets isn't that great of a solution for testing, since it sort of defeats the purpose of having a testing cache if it's just going to expire every 14 days. Beyond the whole everyone who forks it needing to setup a GH secret hassle, you run into the potential problem with the data from ComicVine possibly changing due to editing. I'd lean towards either using a non-expiring testing cache, or uses fixtures for the data. |
That's true, I guess my implementation doesn't work for tests as I'd planned. I'll take a look at my implementation in another PR and make it possible to not expire the cache for tests |