Skip to content
GitLab
Explore
Sign in
Primary navigation
Search or go to…
Project
P
Pulsar
Manage
Activity
Members
Labels
Plan
Issues
Issue boards
Milestones
Iterations
Wiki
Code
Merge requests
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Snippets
Locked files
Build
Pipelines
Jobs
Pipeline schedules
Artifacts
Deploy
Releases
Package Registry
Model registry
Operate
Environments
Terraform modules
Monitor
Incidents
Analyze
Value stream analytics
Contributor analytics
CI/CD analytics
Repository analytics
Code review analytics
Issue analytics
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
NDIP
Pulsar
Commits
239b2201
Unverified
Commit
239b2201
authored
5 years ago
by
John Chilton
Committed by
GitHub
5 years ago
Browse files
Options
Downloads
Plain Diff
Merge pull request #186 from jmchilton/improved-release-linting
Improve readme linting.
parents
e7d09932
bac2cb0b
No related branches found
Branches containing commit
No related tags found
Tags containing commit
No related merge requests found
Changes
4
Hide whitespace changes
Inline
Side-by-side
Showing
4 changed files
.travis.yml
+1
-1
1 addition, 1 deletion
.travis.yml
Makefile
+5
-5
5 additions, 5 deletions
Makefile
dev-requirements.txt
+0
-3
0 additions, 3 deletions
dev-requirements.txt
tox.ini
+4
-4
4 additions, 4 deletions
tox.ini
with
10 additions
and
13 deletions
.travis.yml
+
1
−
1
View file @
239b2201
...
...
@@ -5,7 +5,7 @@ matrix:
-
python
:
2.7
env
:
TOX_ENV=py27-lint
-
python
:
2.7
env
:
TOX_ENV=py27-lint-
readme
env
:
TOX_ENV=py27-lint-
dist
-
python
:
2.7
env
:
TOX_ENV=lint-docs
-
python
:
2.7
...
...
This diff is collapsed.
Click to expand it.
Makefile
+
5
−
5
View file @
239b2201
...
...
@@ -27,7 +27,7 @@ help:
@
echo
"clean-test - remove test and coverage artifacts"
@
echo
"setup-venv - setup a development virutalenv in current directory."
@
echo
"lint - check style with flake8"
@
echo
"lint-
readme - check README formatting for PyPI
"
@
echo
"lint-
dist - twine check dist results, including validating README content
"
@
echo
"lint-docs - check sphinx docs for warnings"
@
echo
"tests - run tests quickly with the default Python"
@
echo
"coverage - check code coverage quickly with the default Python"
...
...
@@ -74,8 +74,8 @@ flake8:
lint
:
$(
IN_VENV
)
tox
-e
py27-lint
&&
tox
-e
py34-lint
lint-
readme
:
$(
IN_VENV
)
python setup.py
check
-r
-s
lint-
dist
:
dist
$(
IN_VENV
)
twine
check
dist/
*
tests
:
$(
IN_VENV
)
nosetests
$(
NOSE_TESTS
)
...
...
@@ -122,7 +122,7 @@ open-rtd: docs
open-project
:
open
$(
PROJECT_URL
)
||
xdg-open
$(
PROJECT_URL
)
dist
:
clean
dist
:
clean
-build clean-pyc
$(
IN_VENV
)
python setup.py sdist bdist_wheel
ls
-l
dist
...
...
@@ -130,7 +130,7 @@ _release-test-artifacts:
$(
IN_VENV
)
twine upload
-r
test
dist/
*
open https://testpypi.python.org/pypi/
$(
PROJECT_NAME
)
||
xdg-open https://testpypi.python.org/pypi/
$(
PROJECT_NAME
)
dist-all
:
dist _dist-lib
dist-all
:
lint-
dist _dist-lib
release-test-artifacts
:
dist-all _release-test-artifacts
...
...
This diff is collapsed.
Click to expand it.
dev-requirements.txt
+
0
−
3
View file @
239b2201
...
...
@@ -16,9 +16,6 @@ sphinx==1.2
pyflakes
flake8
# Used to check readme.
readme
# For release
wheel
twine
This diff is collapsed.
Click to expand it.
tox.ini
+
4
−
4
View file @
239b2201
[tox]
envlist
=
py27-lint, py27-lint-
readme
, lint-docs, py35-lint, py27, py34, py27-unit, py34-unit
envlist
=
py27-lint, py27-lint-
dist
, lint-docs, py35-lint, py27, py34, py27-unit, py34-unit
toxworkdir
=
{env:TOX_WORK_DIR:.tox}
[testenv]
...
...
@@ -37,12 +37,12 @@ commands = flake8 --ignore W504 pulsar test
skip_install
=
True
deps
=
flake8
[testenv:py27-lint-
readme
]
commands
=
make lint-
readme
[testenv:py27-lint-
dist
]
commands
=
make lint-
dist
skip_install
=
True
whitelist_externals
=
make
deps
=
readm
e
twin
e
[testenv:lint-docs]
commands
=
make lint-docs
...
...
This diff is collapsed.
Click to expand it.
Preview
0%
Loading
Try again
or
attach a new file
.
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Save comment
Cancel
Please
register
or
sign in
to comment