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

Convert to entrypoint plugin style + modernize #25

Merged
merged 6 commits into from
Jul 6, 2024
Merged

Conversation

dgw
Copy link
Owner

@dgw dgw commented Jul 5, 2024

  • Replaced setup.py metadata with pyproject.toml
  • Moved plugin code from sopel_modules.wolfram into sopel_wolfram and updated packaging instructions accordingly
  • Switched sopel.modulesopel.plugin
  • Upgraded from Sopel 7.x to 8.x
  • Upgraded from wolframalpha 3.x to 5.x
  • Replaced NEWS and README.rst with Markdown equivalents
  • Miscellaneous housekeeping (unused tests folder, obsolete release script, configure() prompts, etc.)

* Replaced `setup.py` metadata with `pyproject.toml`
* Moved plugin code from `sopel_modules.wolfram` into `sopel_wolfram`
  and updated packaging instructions accordingly
* Modernized a few things to be compatible with Sopel 8.x (and dropped
  support for 7.0; 7.1+ only now because of `sopel.plugin`)
* Replaced NEWS and README.rst with Markdown equivalents
* Miscellaneous housekeeping (unused `tests` folder, obsolete release
  script, etc.)
@dgw dgw added the packaging label Jul 5, 2024
@dgw dgw added this to the v0.6.0 "Bocchi de Skip" milestone Jul 5, 2024
@dgw
Copy link
Owner Author

dgw commented Jul 6, 2024

Installs fine locally (pip install -e .). Testing is stalled because I can't get an AppID out of Wolfram. Either their developer portal is broken, or my account is messed up. An old AppID I pulled from some JSON it fetches (via developer tools) returns an API error that the AppID is not valid, and I've run out of ideas for tonight.

@dgw
Copy link
Owner Author

dgw commented Jul 6, 2024

I got past the testing roadblock with a fresh Wolfram account. Mildly annoying that I now have two (they don't support deletions), but at least I didn't have to contact support and wait for a reply over the holiday weekend.

@dgw dgw merged commit 5456298 into master Jul 6, 2024
@dgw dgw deleted the entrypoint-ify branch July 6, 2024 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant