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

Rendering with current markdown version #571

Closed
apeltzer opened this issue Feb 29, 2020 · 9 comments
Closed

Rendering with current markdown version #571

apeltzer opened this issue Feb 29, 2020 · 9 comments
Assignees
Labels
bug Something isn't working

Comments

@apeltzer
Copy link
Member

When using the latest (V1.9) template, we replaced the R code with a Python script. This is broken when updating the markdown package in the container used for any pipeline and results in e.g. errors like this:

  Traceback (most recent call last):
    File "/Users/alexanderpeltzer/Downloads/eager/bin/markdown_to_html.py", line 100, in <module>
      sys.exit(main())
    File "/Users/alexanderpeltzer/Downloads/eager/bin/markdown_to_html.py", line 95, in main
      converted_md = convert_markdown(args.mdfile.name)
    File "/Users/alexanderpeltzer/Downloads/eager/bin/markdown_to_html.py", line 28, in convert_markdown
      'title': 'Table of Contents'
    File "/opt/conda/envs/nf-core-eager-2.1.0dev/lib/python3.7/site-packages/markdown/core.py", line 387, in markdown
      md = Markdown(**kwargs)
    File "/opt/conda/envs/nf-core-eager-2.1.0dev/lib/python3.7/site-packages/markdown/core.py", line 97, in __init__
      configs=kwargs.get('extension_configs', {}))
    File "/opt/conda/envs/nf-core-eager-2.1.0dev/lib/python3.7/site-packages/markdown/core.py", line 125, in registerExtensions
      ext._extendMarkdown(self)
    File "/opt/conda/envs/nf-core-eager-2.1.0dev/lib/python3.7/site-packages/markdown/extensions/__init__.py", line 77, in _extendMarkdown
      self.extendMarkdown(md)
    File "/opt/conda/envs/nf-core-eager-2.1.0dev/lib/python3.7/site-packages/pymdownx/extra.py", line 60, in extendMarkdown
      md.registerExtensions(extra_extensions, self.config)
    File "/opt/conda/envs/nf-core-eager-2.1.0dev/lib/python3.7/site-packages/markdown/core.py", line 125, in registerExtensions
      ext._extendMarkdown(self)
    File "/opt/conda/envs/nf-core-eager-2.1.0dev/lib/python3.7/site-packages/markdown/extensions/__init__.py", line 77, in _extendMarkdown
      self.extendMarkdown(md)
    File "/opt/conda/envs/nf-core-eager-2.1.0dev/lib/python3.7/site-packages/pymdownx/extrarawhtml.py", line 30, in extendMarkdown
      extra.MarkdownInHtmlProcessor(md.parser), 'markdown_block', 105
  AttributeError: module 'markdown.extensions.extra' has no attribute 'MarkdownInHtmlProcessor'
@apeltzer apeltzer added the bug Something isn't working label Feb 29, 2020
@apeltzer
Copy link
Member Author

Not directly related to this here #391 but could be fixed in the same go ;-)

@phue
Copy link
Member

phue commented Apr 9, 2020

Just for reference:
This issue was fixed in facelessuser/pymdown-extensions@f17ba7b#diff-1915e1d5e453671c14bd0dd5ab7a2b1f
So once pymdown-extensions 7.0.0 hits the conda-forge repos it should also work with newer markdown versions

@ewels
Copy link
Member

ewels commented Apr 9, 2020

Nice, good find @phue - if they are depreciating something does that mean that we should update a method we're using somewhere? Though I think we're not actually using that method, so I guess it's only a cross-package compatibility thing and we're fine..

@ewels
Copy link
Member

ewels commented Apr 9, 2020

Hmm, looks like the feedstock for this package is a little out of love: https://github.com/conda-forge/pymdown-extensions-feedstock/pulls

@apeltzer
Copy link
Member Author

apeltzer commented Apr 9, 2020

Yeah :-(

@ewels
Copy link
Member

ewels commented Apr 9, 2020

Have bumped the version and re-rendered with conda smithy. Hopefully that will fix it. At least the Azure builds will still be there if it fails so that we can see what is going wrong..

conda-forge/pymdown-extensions-feedstock#5

@ewels
Copy link
Member

ewels commented Apr 9, 2020

ok, tests are passing so it should be good to merge ✅ - just needs someone to merge it now 🤞

@ewels
Copy link
Member

ewels commented Apr 15, 2020

I pinged the conda-forge core team and got myself added as a maintainer, so the above PR was merged and hopefully we can move this forward now..

@drpatelh
Copy link
Member

drpatelh commented Jul 6, 2020

This should be fixed now.

@drpatelh drpatelh closed this as completed Jul 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants