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.

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

Convert scripts in bin to console_scripts #224

Merged
merged 6 commits into from
Jul 16, 2016
Merged

Conversation

rodrigc
Copy link
Contributor

@rodrigc rodrigc commented Jun 26, 2016

@codecov-io
Copy link

codecov-io commented Jun 26, 2016

Current coverage is 90.15%

Merging #224 into trunk will increase coverage by 1.53%

@@              trunk       #224   diff @@
==========================================
  Files           821        821           
  Lines        143746     143709     -37   
  Methods           0          0           
  Messages          0          0           
  Branches      13472      13468      -4   
==========================================
+ Hits         127390     129567   +2177   
+ Misses        13959      11895   -2064   
+ Partials       2397       2247    -150   

Powered by Codecov. Last updated by 2dd6d50...a001904

@alex
Copy link
Member

alex commented Jun 26, 2016

looks like some of the builders need to be updated

@rodrigc
Copy link
Contributor Author

rodrigc commented Jun 27, 2016

@adiroiban: why is codecov showing 71.42% coverage here: https://codecov.io/gh/twisted/twisted/compare/8dd2292cac9be85a9f81aef336c35bdb89f46e1c...87cb6c0d9af66591d218bd53684d58e15601daf4

It looks like it is not reporting coverage of the _PY3 block. Isn't the https://buildbot.twistedmatrix.com/builders/fedora22-py3.4-coverage build uploading coverage results to codecov.io ?

@@ -43,7 +43,9 @@ def main(args):
packages=setuptools.find_packages(),
install_requires=requirements,
conditionalExtensions=getExtensions(),
scripts=getScripts(),
entry_points={
'console_scripts': getConsoleScripts()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the wrong indentation.

@rodrigc rodrigc force-pushed the 8491-rodrigc-console branch 8 times, most recently from 9e8826d to f8c80dc Compare July 15, 2016 18:14
twisted.python.dist.getScripts() was removed during
the conversion to the console_scripts entry point.
These scripts are now installed by the console_scripts
entry point in setup.py.
@rodrigc rodrigc merged commit c5a4c63 into trunk Jul 16, 2016
@rodrigc rodrigc deleted the 8491-rodrigc-console branch July 16, 2016 03:59
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.

None yet

3 participants