Skip to content

remove remnants of python package #96

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

Merged
merged 1 commit into from
Sep 1, 2022
Merged

remove remnants of python package #96

merged 1 commit into from
Sep 1, 2022

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cleanup cpp_linter source code starting from v2
2 participants