-
-
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
Add Touch as namespace plugin #2
Conversation
@wombelix: You can ignore the failed Lint job — that is happening because the PyPI package does not exist yet, so obviously the link to its PyPI page is broken. This will be resolved once the package has been published. The test failure seen here seems to occur because we are running tests even though there currently aren't any tests. Do you think you could add at least one test, even if it is simple, in order to resolve that problem? |
Deprecation warnings are now only displayed when `--deprecations` flag is passed.
Otherwise CI fails until the package is first published to PyPI.
62775e6
to
804964e
Compare
@wombelix: I am using this Touch plugin migration endeavor as a test case for switching to Ruff, replacing a bunch of slow & fussy code linters in the process. Would you have a look and let me know if you have any comments? Otherwise I will plan to merge this on Friday, 28 April. Cheers! |
@wombelix: Many thanks for your assistance in migrating this plugin. Much appreciated! ✨ |
Migrated from https://github.com/getpelican/pelican-plugins/tree/master/touch and subsequently updated by @wombelix and @justinmayer.