FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Random bits by sverhoeven · Pull Request #287 · NLeSC/python-template · GitHub

Random bits - #287

Merged
sverhoeven merged 43 commits into
mainfrom
kitchen-sink
May 7, 2021
Merged

Random bits#287
sverhoeven merged 43 commits into
mainfrom
kitchen-sink

Conversation

sverhoeven commented May 4, 2021
edited
Loading

Copy link
Copy Markdown
Member

Random bits to make template & package developer experience better.

Added

  • Template unit tests for documentation generation, linting and version bumping
  • Docstring for function
  • Coverage and doctest commands for documentation #97

Changed

  • Initial linting is error free #227
  • Consolidated test/lint/build/docs into single matrix workflow #270
  • Enforce isort configuration
  • Default for package_short_description in cookiecutter questionnaire
  • Link checker ignores GH private pages and test pypi site #288
  • In CI build workflow make prospector die if there are errors #275
  • All example tests make use of example function
  • Use bumpversion for version in Sphinx config #44
  • Regenerated docs/conf.py with sphinx-quickstart v3.5.4 + enabled built-in extensions #44
  • Generate api rst files with extension instead of custom function #95

Removed

  • .pylintrc file, was too strict, too soon #267
  • Unused development dependencies #167
  • Statements in project_setup.md already mentioned in README.dev.md

Tested with https://github.com/sverhoeven/justatest2, where workflows are build workflow is green and https://justatest2.readthedocs.io/en/latest/autoapi/justatest2/my_module/index.html works.

sverhoeven marked this pull request as ready for review May 4, 2021 07:17
sverhoeven changed the title Kitchen sink Random bits May 4, 2021
Comment thread tests/test_project.py
Comment thread tests/test_project.py Outdated
Co-authored-by: Faruk D. <fdiblen@users.noreply.github.com>

fdiblen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Amazing contribution. Thank you @sverhoeven 🎖️

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL