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

Contrib folder update #167

Merged
merged 7 commits into from
Dec 29, 2022
Merged

Conversation

anderskaplan
Copy link
Contributor

This PR is a proposed solution for #101, Make renderers from the "contrib" folder easier to use.
The approach taken is as described in the discussion thread on that issue: basically to move the contrib directory inside the mistletoe source directory.

In addition to moving files around, the PR also includes some documentation updates to explain the use of the contrib folder and how to use the renderers in the contrib directory from the CLI.

NOTE the mistletoe.egg-info folder has not been updated, as the egg format seems to be quite obsolete by now and superseeded by wheel. I'd recommend to remove the egg-info folder.

Copy link

@tucked tucked left a comment

Choose a reason for hiding this comment

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

Woohoo 🎉 LGTM! Thanks for this 👍

@pbodnar
Copy link
Collaborator

pbodnar commented Dec 5, 2022

NOTE the mistletoe.egg-info folder has not been updated, as the egg format seems to be quite obsolete by now and superseeded by wheel. I'd recommend to remove the egg-info folder.

Well, the ".egg-info" folder is not versioned and it seems to be generated when installing mistletoe via pip3 install -e .. So no problem here, actually? :)

@pbodnar
Copy link
Collaborator

pbodnar commented Dec 5, 2022

... Added init.py to make the contrib directory a python module. ...

I guess this is OK. I have found that the __init__.py file is no longer necessary since Python 3.3, yet it seems to be recommended for "regular packages", like this one - as described at SO?

Copy link
Collaborator

@pbodnar pbodnar left a comment

Choose a reason for hiding this comment

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

@anderskaplan, thank you! I think it looks fine, just some smaller amendments left. :)

Also, I expect that I can probably squash all the commits into one on merge, while adding issue link (#101) to the commit message, OK?

CONTRIBUTING.md Outdated Show resolved Hide resolved
@anderskaplan
Copy link
Contributor Author

anderskaplan commented Dec 7, 2022

Comments addressed, should be good to go now!

  • The ".egg-info" folder is not versioned: yeah you're right. It's not a problem that the file list needs to be updated, then. Good.
  • Agree that the recommendation seems to be to keep the init for packages.
  • Updates applied to the README: the ones you pointed out plus a few more that I spotted when searching for "contrib".
  • Squash merge is fine with me.

@pbodnar
Copy link
Collaborator

pbodnar commented Dec 17, 2022

@anderskaplan, LGTM, thank you!

I just have to prepare a PR for changing https://github.com/hluk/copyq-commands/blob/master/Global/convert-markdown.ini which uses the renderers from the contrib folder, and I will merge this afterwards.

Also, what do you think about including this "breaking change" (not much we can do to keep it backwards compatible at least for a while, I guess) into version 1.0.0 - yes, or no? :)

@pbodnar pbodnar added the breaking-change <https://en.wikipedia.org/wiki/Backward_compatibility> label Dec 17, 2022
@anderskaplan
Copy link
Contributor Author

Hi @pbodnar, as you say there's not much that can be done to keep this backward compatible, so I thinks its better to rip off the band-aid as soon as possible and be done with it. Besides, the label "contrib" indicates that the software is more experimental and that a lower standard of backward compatibility is to be expected.

pbodnar added a commit to pbodnar/copyq-commands that referenced this pull request Dec 23, 2022
…toe project

mistletoe gets new updates in its original Git repository again,
so we can safely switch to that one now.

This also means using updated paths to mistletoe's renderers
from the "contrib" folder (see miyuchina/mistletoe#167).
@anderskaplan
Copy link
Contributor Author

Rebased to resolve a merge conflict.

@pbodnar pbodnar merged commit 29d461f into miyuchina:master Dec 29, 2022
@pbodnar
Copy link
Collaborator

pbodnar commented Dec 29, 2022

Hi @anderskaplan, thank you, so I have merged it right away. :)

hluk pushed a commit to hluk/copyq-commands that referenced this pull request Dec 30, 2022
…toe project (#62)

mistletoe gets new updates in its original Git repository again,
so we can safely switch to that one now.

This also means using updated paths to mistletoe's renderers
from the "contrib" folder (see miyuchina/mistletoe#167).
@pbodnar pbodnar mentioned this pull request Jan 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change <https://en.wikipedia.org/wiki/Backward_compatibility>
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants