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

TypeError: strptime() argument 1 must be str, not datetime.datetime #290

Closed
ZhiyuanChen opened this issue Jun 11, 2024 · 2 comments · Fixed by #300
Closed

TypeError: strptime() argument 1 must be str, not datetime.datetime #290

ZhiyuanChen opened this issue Jun 11, 2024 · 2 comments · Fixed by #300
Labels
bug Something isn't working

Comments

@ZhiyuanChen
Copy link

It seems 1.13.0 introduces a few new bugs

My pipeline worked fine, but is complaining now.

https://github.com/DLS5-Omics/multimolecule/actions/runs/9465496343/job/26075394011
ZhiyuanChen added a commit to ZhiyuanChen/mkdocs-rss-plugin that referenced this issue Jun 11, 2024
@ZhiyuanChen
check default_time before parsing
7b648f4 
fix Guts#290

Signed-off-by: Zhiyuan Chen <[email protected]>
@ZhiyuanChen ZhiyuanChen mentioned this issue Jun 11, 2024
Closed
@Guts
Copy link
Owner

Guts commented Jun 23, 2024

Hey @ZhiyuanChen

Is your issue still active? Have you seen my comment on your PR #291 (comment)?

@Guts Guts added the bug Something isn't working label Jun 23, 2024
@Guts
Copy link
Owner

Guts commented Jun 23, 2024

I investigated a little further because the error seems strange to me since the default value of date_from_meta.default_time which is used in your configuration.
After looking at your build in verbose mode, it appears that the doc is built 2 times.

It seems that the problem comes from the i18n plugin. By deactivating it, the build goes smoothly.

Guts added a commit that referenced this issue Jun 23, 2024
@Guts
fix(config): do not raise error for bad date_from_meta.default_time v…
38c477f 
…alue and fallback to default

Closes #290
@Guts Guts mentioned this issue Jun 23, 2024
Merged
Guts added a commit that referenced this issue Jun 23, 2024
@Guts
fix(config): do not raise error for bad date_from_meta.default_time v…
85ef616 
…alue and fallback to default (#300)

Closes #290
@Guts Guts closed this as completed in #300 Jun 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
2 participants
@Guts @ZhiyuanChen