Skip to content

Conversation

@2bndy5
Copy link
Collaborator

@2bndy5 2bndy5 commented Sep 1, 2022

solves #95

  • altered docs
    • added the new logo & favicon.
    • changed colors for readability
    • removed API reference pages
    • adds contributing guidelines page
  • removed python sources
  • removed unit tests
  • removed requirements*.txt
  • removed gitpod config because
    it was only related to installing python
    dependencies (specifically not for docs)
  • removed pre-commit hooks related to python sources
  • removed pytest/coverage workflows
  • updated README
    • removed coverage badge
    • changed target for docs badge to point to hosted docs
  • reduced mkdocs config/requirements accordingly
  • changed link to LICENSE to be absolute
    because it was broken from the docs site
  • adjust mkdocs CI accordingly
  • adjust test repo triggers in run-test.yml CI workflow
    The package no longer exists on the branch, so running the test repo to install a python package from here will always fail.
@2bndy5 2bndy5 linked an issue Sep 1, 2022 that may be closed by this pull request
@2bndy5 2bndy5 force-pushed the remove-python-stuff branch from cb40c99 to 8c591bd Compare September 1, 2022 10:17
solves #95 - altered docs - added the new logo & favicon. - changed colors for readability - removed API reference pages - adds contributing guidelines page - removed python sources - removed unit tests - removed requirements*.txt - removed gitpod config because it was only related to installing python dependencies (specifically not for docs) - removed pre-commit hooks related to python sources - removed pytest/coverage workflows - updated README - removed coverage badge - changed target for docs badge to point to hosted docs - reduced mkdocs config/requirements accordingly - changed link to LICENSE to be absolute because it was broken from the docs site - adjust mkdocs CI - adjust test-repo triggers in run-test.yml CI workflow
@2bndy5 2bndy5 force-pushed the remove-python-stuff branch from 8c591bd to 90ec2f7 Compare September 1, 2022 10:24
@2bndy5 2bndy5 requested a review from shenxianpeng September 1, 2022 10:27
Copy link
Collaborator

@shenxianpeng shenxianpeng left a comment

Choose a reason for hiding this comment

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

LGTM

@2bndy5 2bndy5 merged commit 49c2eb9 into main Sep 1, 2022
@2bndy5 2bndy5 deleted the remove-python-stuff branch September 1, 2022 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants