-
Notifications
You must be signed in to change notification settings - Fork 185
fix: rename TestColumn classes to MockColumn to fix pytest collection warnings #1200
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
Open
hashwnath
wants to merge
15
commits into
capitalone:dev
Choose a base branch
from
hashwnath:fix/pytest-collection-warnings
base: dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
4545841
staging/main/0.12.0 (#1145)
taylorfturner 50da93f
Staging release 0.13.0 (#1165) (#1166)
armaan-dhillon c0f86fb
chore: update mypy dependencies (#1171)
shania-m 48d0fd5
chore: refactor release process (#1170)
shania-m aafeef6
chore: Update publish-package.yml (#1172)
shania-m ed8188a
gh pages update (#1173)
shania-m 09c2062
Update gh-pages workflows (#1174)
shania-m 9919af4
Remove Versions from Intro page (#1175)
shania-m 5a1ee0a
chore: Workflow permissions (#1182)
shania-m 1a84430
docs: add architecture.rst for algorithm rationale, testing, versioni…
mhmotamedi c035a05
ops: drop python 3.9 & resolve vulns (#1192)
ryanSoley ebb7ee1
ops: properly configure whitesource (#1195)
ryanSoley 5a381d5
fix: use core config (#1198)
ryanSoley 60256a3
fix: rename TestColumn classes to MockColumn to fix pytest warnings
hashwnath ed03bf2
Merge remote-tracking branch 'origin/dev' into fix/pytest-collection-…
hashwnath File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,47 @@ | ||
| # This workflow builds and publishes the latest docs to | ||
| # the `gh-pages` branch. | ||
| # For more details: https://github.com/marketplace/actions/deploy-to-github-pages | ||
| name: Publish docs | ||
|
|
||
| on: | ||
| release: | ||
| types: [created] | ||
| workflow_dispatch: | ||
|
|
||
| jobs: | ||
| build-and-deploy: | ||
| runs-on: ubuntu-latest | ||
| permissions: | ||
| contents: write | ||
| pages: write | ||
| defaults: | ||
| run: | ||
| shell: bash -l {0} | ||
| steps: | ||
| - uses: actions/checkout@v2 | ||
| with: | ||
| # fetch all tags so `versioneer` can properly determine current version | ||
| fetch-depth: 0 | ||
| - name: Set up Python | ||
| uses: actions/setup-python@v2 | ||
| with: | ||
| python-version: '3.11' | ||
| - name: Install pandoc | ||
| uses: pandoc/actions/setup@v1 | ||
| - name: Install dependencies | ||
| run: | | ||
| pip install -r requirements.txt | ||
| pip install -r requirements-ml.txt | ||
| pip install -r requirements-reports.txt | ||
| pip install -r requirements-docs.txt | ||
| pip install -e . | ||
|
|
||
| - name: Build | ||
| run: | | ||
| cd _docs/docs | ||
| python update_documentation.py | ||
| - name: Publish | ||
| uses: JamesIves/github-pages-deploy-action@v4 | ||
| with: | ||
| branch: gh-pages | ||
| folder: _docs/docs/LATEST/html |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,52 @@ | ||
| # This workflow publishes the package to pypi. | ||
| # For more details: | ||
| # https://docs.github.com/en/actions/guides/building-and-testing-python#publishing-to-package-registries | ||
| name: Publish to PyPi | ||
|
|
||
| on: | ||
| release: | ||
| types: [created] | ||
| workflow_dispatch: | ||
|
|
||
| jobs: | ||
| deploy: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v2 | ||
| # fetch all tags so `versioneer` can properly determine current version | ||
| with: | ||
| fetch-depth: 0 | ||
| - name: Check if current commit is tagged | ||
| # fails and cancels release if the current commit is not tagged | ||
| run: | | ||
| git describe --exact-match --tags | ||
| - name: Set up Python | ||
| uses: actions/setup-python@v2 | ||
| with: | ||
| python-version: '3.11' | ||
| - name: Install dependencies | ||
| run: | | ||
| python -m pip install --upgrade pip | ||
| if [ -f requirements.txt ]; then pip install -r requirements.txt; fi | ||
| if [ -f requirements-ml.txt ]; then pip install -r requirements-ml.txt; fi | ||
| if [ -f requirements-reports.txt ]; then pip install -r requirements-reports.txt; fi | ||
| pip install setuptools wheel twine | ||
| - name: Build | ||
| env: | ||
| TWINE_USERNAME: ${{ secrets.PYPI_USERNAME }} | ||
| TWINE_PASSWORD: ${{ secrets.PYPI_PASSWORD }} | ||
| TWINE_REPOSITORY: pypi | ||
| run: | | ||
| python setup.py sdist bdist_wheel | ||
| - name: Test build | ||
| # fails and cancels release if the built package fails to import | ||
| run: | | ||
| pip install dist/*.whl | ||
| python -c 'import dataprofiler; print(dataprofiler.__version__)' | ||
| - name: Publish | ||
| env: | ||
| TWINE_USERNAME: ${{ secrets.PYPI_USERNAME }} | ||
| TWINE_PASSWORD: ${{ secrets.PYPI_PASSWORD }} | ||
| TWINE_REPOSITORY: pypi | ||
| run: | | ||
| twine upload dist/* |
This file was deleted.
Oops, something went wrong.
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| { | ||
| "settingsInheritedFrom": "capitalone/whitesource-config" | ||
| } |
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,59 @@ | ||
| Visit our [documentation page.](https://capitalone.github.io/DataProfiler) | ||
|
|
||
| ### How to properly write documentation: | ||
|
|
||
| #### Packages | ||
| In any package directory, overall package comments can be made in the | ||
| \_\_init\_\_.py of the directory. At the top of the \_\_init\_\_.py, | ||
| include your comments in between triple quotations. | ||
|
|
||
| #### Classes | ||
| In any class file, include overall class comments at the top of the file | ||
| in between triple quotes and/or in the init function. | ||
|
|
||
| #### Functions | ||
| reStructuredText Docstring Format is the standard. Here is an example: | ||
|
|
||
| def format_data(self, predictions, verbose=False): | ||
| """ | ||
| Formats word level labeling of the Unstructured Data Labeler as you want | ||
|
|
||
| :param predictions: A 2D list of word level predictions/labeling | ||
| :type predictions: Dict | ||
| :param verbose: A flag to determine verbosity | ||
| :type verbose: Bool | ||
| :return: JSON structure containing specified formatted output | ||
| :rtype: JSON | ||
|
|
||
| :Example: | ||
| Look at this test. Don't forget the double colons to make a code block:: | ||
| This is a codeblock | ||
| Type example code here | ||
| """ | ||
|
|
||
| ### How to update the documentation: | ||
|
|
||
|
|
||
| 1. Set up your local environment | ||
| ```bash | ||
| # install sphinx requirements | ||
| # install the requirements from the feature branch | ||
| pip install pandoc && | ||
| pip install -r requirements.txt && | ||
| pip install -r requirements-ml.txt && | ||
| pip install -r requirements-reports.txt && | ||
| pip install -r requirements-docs.txt && | ||
| pip install -e . | ||
|
|
||
| ``` | ||
| 2. And finally, from the root of `DataProfiler`, run the following commands to generate the sphinx documentation: | ||
| ```bash | ||
| cd _docs/docs | ||
| python update_documentation.py | ||
|
|
||
| ``` | ||
|
|
||
| 3. View new docs | ||
| ```bash | ||
| open index.html | ||
| ``` |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,20 @@ | ||
| # Minimal makefile for Sphinx documentation | ||
| # | ||
|
|
||
| # You can set these variables from the command line, and also | ||
| # from the environment for the first two. | ||
| SPHINXOPTS ?= | ||
| SPHINXBUILD ?= sphinx-build | ||
| SOURCEDIR = source | ||
| BUILDDIR = buildcode | ||
|
|
||
| # Put it first so that "make" without argument is like "make help". | ||
| help: | ||
| @$(SPHINXBUILD) -M help "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O) | ||
|
|
||
| .PHONY: help Makefile | ||
|
|
||
| # Catch-all target: route all unknown targets to Sphinx using the new | ||
| # "make mode" option. $(O) is meant as a shortcut for $(SPHINXOPTS). | ||
| %: Makefile | ||
| @$(SPHINXBUILD) -M $@ "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O) |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,35 @@ | ||
| @ECHO OFF | ||
|
|
||
| pushd %~dp0 | ||
|
|
||
| REM Command file for Sphinx documentation | ||
|
|
||
| if "%SPHINXBUILD%" == "" ( | ||
| set SPHINXBUILD=sphinx-build | ||
| ) | ||
| set SOURCEDIR=source | ||
| set BUILDDIR=build | ||
|
|
||
| if "%1" == "" goto help | ||
|
|
||
| %SPHINXBUILD% >NUL 2>NUL | ||
| if errorlevel 9009 ( | ||
| echo. | ||
| echo.The 'sphinx-build' command was not found. Make sure you have Sphinx | ||
| echo.installed, then set the SPHINXBUILD environment variable to point | ||
| echo.to the full path of the 'sphinx-build' executable. Alternatively you | ||
| echo.may add the Sphinx directory to PATH. | ||
| echo. | ||
| echo.If you don't have Sphinx installed, grab it from | ||
| echo.http://sphinx-doc.org/ | ||
| exit /b 1 | ||
| ) | ||
|
|
||
| %SPHINXBUILD% -M %1 %SOURCEDIR% %BUILDDIR% %SPHINXOPTS% %O% | ||
| goto end | ||
|
|
||
| :help | ||
| %SPHINXBUILD% -M help %SOURCEDIR% %BUILDDIR% %SPHINXOPTS% %O% | ||
|
|
||
| :end | ||
| popd |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this necessary