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

Bundle "ninja_syntax.py" into the wheel #1

Closed
wants to merge 1 commit into from
Closed

Conversation

jcfr
Copy link
Contributor

@jcfr jcfr commented Dec 22, 2016

To maintain backward compatibility with existing ninja wheels, this
commit includes "ninja_syntax.py".

Suggested-by: Marat Dukhan maratek@gmail.com

@Maratyszcza
Copy link

LGTM, please merge before you upload the PyPI package.

@jcfr
Copy link
Contributor Author

jcfr commented Dec 23, 2016

@Maratyszcza Will do. And will make sure the CI is green before doing so.

Note also that the next version of the package will be 1.7.2 so that it matches the version of ninja.

To maintain backward compatibility with existing ninja wheels, this
commit includes "ninja_syntax.py".

Suggested-by: Marat Dukhan <maratek@gmail.com>
@codecov-io
Copy link

codecov-io commented Dec 24, 2016

Current coverage is 86.36% (diff: 83.33%)

Merging #1 into master will decrease coverage by 1.13%

@@             master         #1   diff @@
==========================================
  Files             1          1          
  Lines            16         22     +6   
  Methods           0          0          
  Messages          0          0          
  Branches          2          3     +1   
==========================================
+ Hits             14         19     +5   
  Misses            0          0          
- Partials          2          3     +1   

Powered by Codecov. Last update 7b1af3e...147e5df

@jcfr
Copy link
Contributor Author

jcfr commented Dec 24, 2016

Since Appveyor, CircleCI and Travis were green. Topic was integrated as ee84aed

The CI looks red only because codecov complains

@jcfr jcfr closed this Dec 24, 2016
@jcfr jcfr deleted the add-ninja_syntax branch December 24, 2016 02:29
@jcfr
Copy link
Contributor Author

jcfr commented Dec 24, 2016

@jcfr
Copy link
Contributor Author

jcfr commented Dec 24, 2016

Associate packages will be uploaded as soon Appveyor, CircleCI and Travis are done generating them.

@jcfr
Copy link
Contributor Author

jcfr commented Dec 24, 2016

Et voila, see https://pypi.python.org/pypi/ninja/1.7.2

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