Skip to content
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.

Sign up for GitHub

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

Jump to bottom

docs: remove the need for special scripts #1914

Merged
merged 2 commits into from
Sep 5, 2016
Merged

docs: remove the need for special scripts #1914

merged 2 commits into from
Sep 5, 2016

Conversation

RonnyPfannschmidt
Copy link
Member

  • import version from pytest since it is importable
  • remove makefile bits from legacy hosting
  • remove the script to determine the version for legacy hosting

@RonnyPfannschmidt
docs: remove the need for special scripts
e21ae39 
* import version from pytest since it is importable
* remove makefile bits from legacy hosting
* remove the script to determine the version for legacy hosting
@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.078% when pulling e21ae39 on RonnyPfannschmidt:docs-remove-getdoctarget into 8a41b26 on pytest-dev:master.

@The-Compiler
Copy link
Member

So do we still need _getdoctarget.py?

@RonnyPfannschmidt
Copy link
Member Author

whops, i did rm instead of git rm

@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.078% when pulling a2b8981 on RonnyPfannschmidt:docs-remove-getdoctarget into 8a41b26 on pytest-dev:master.

@nicoddemus
Copy link
Member

Thanks @RonnyPfannschmidt!

@nicoddemus nicoddemus merged commit 99efc28 into pytest-dev:master Sep 5, 2016
@RonnyPfannschmidt RonnyPfannschmidt deleted the docs-remove-getdoctarget branch October 24, 2016 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@RonnyPfannschmidt @coveralls @The-Compiler @nicoddemus