Skip to content

Commit e66ed36

Browse files
authored
Fix errors in README and add a test to avoid such errors in the future (#1103)
* fixing pypi readme errors and adding a test Signed-off-by: Amit Sharma <[email protected]> * installing dynamic versioning Signed-off-by: Amit Sharma <[email protected]> --------- Signed-off-by: Amit Sharma <[email protected]>
1 parent e7bd2ad commit e66ed36

File tree

2 files changed

+10
-0
lines changed

2 files changed

+10
-0
lines changed

.github/workflows/ci.yml

+8
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,14 @@ jobs:
7070
- name: Test EconML based functionality
7171
run: poetry run poe test_econml
7272

73+
- name: Test README errors for PyPI
74+
run: |
75+
pip install poetry-dynamic-versioning
76+
pip install twine
77+
poetry-dynamic-versioning
78+
poetry build
79+
twine check dist/*
80+
7381
- name: Notify Discord
7482
if: failure() && github.ref == 'refs/heads/main'
7583
uses: th0th/[email protected]

README.rst

+2
Original file line numberDiff line numberDiff line change
@@ -102,8 +102,10 @@ If you face "Solving environment" problems with conda, then try :code:`conda upd
102102
If you prefer to use the latest dev version, your dependency management tool will need to point at our GitHub repository.
103103

104104
.. code:: shell
105+
105106
pip install git+https://github.com/py-why/dowhy@main
106107
108+
107109
**Requirements**
108110

109111
DoWhy requires a few dependencies.

0 commit comments

Comments
 (0)