Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

updated documentation#5

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
filmor merged 10 commits intomasterfromdenfromufa-documentation
Sep 14, 2016
Merged

updated documentation#5

filmor merged 10 commits intomasterfromdenfromufa-documentation
Sep 14, 2016

Conversation

den-run-ai
Copy link
Contributor

No description provided.

@den-run-ai
Copy link
ContributorAuthor

den-run-ai commentedSep 13, 2016
edited
Loading

@filmor@vmuriart@tonyroberts please review documentation update. The motivation was outdated information on python guide referencing these docs:

https://github.com/kennethreitz/python-guide/pull/746

@tonyroberts
Copy link
Contributor

👍

@filmor
Copy link
Member

Just one small thing: Do we really want people to usenpython? Does that make sense at all, seeing that it's completely equivalent to runningpython and then importingclr?

@den-run-ai
Copy link
ContributorAuthor

@filmor npython is still useful for quickly troubleshooting the embedding cases:

https://github.com/pythonnet/pythonnet/search?q=npython&type=Issues

Also@cgohlke still bundles npython.exe in his wheels:

http://www.lfd.uci.edu/~gohlke/pythonlibs/#pythonnet

@tonyroberts
Copy link
Contributor

@denfromufa I agree with@filmor that it's better for new users to be pointed to the regular python instead of nPython, otherwise they might get the impression somehow using pythonnetrequires npython. What about changing it to python, and adding a note that nPython is available as an example of embedding the CPython interpreter in a c# application?

@den-run-ai
Copy link
ContributorAuthor

Agreed! Let me change the purpose of nPython.exe in these docs from "using
pythonnet" to "example of embedding Python in .NET". The primary way to use
pythonnet should be "import clr" in python/ipython interpreter.

On Tue, Sep 13, 2016 at 10:03 AM, Tony Robertsnotifications@github.com
wrote:

@denfromufahttps://github.com/denfromufa I agree with@filmor
https://github.com/filmor that it's better for new users to be pointed
to the regular python instead of nPython, otherwise they might get the
impression somehow using pythonnetrequires npython. What about
changing it to python, and adding a note that nPython is available as an
example of embedding the CPython interpreter in a c# application?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#5 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHgZ5S9gdlOhAIFP7-NIc2HlVDtTG-vPks5qprtJgaJpZM4J7QJD
.

@den-run-ai
Copy link
ContributorAuthor

@tonyroberts@filmor done! please go through a second round of review.

@filmor
Copy link
Member

Looks good to me.

@tonyroberts
Copy link
Contributor

me too

@filmor
Copy link
Member

Then squash and merge :)

@filmorfilmor merged commit27d3982 intomasterSep 14, 2016
@filmorfilmor deleted the denfromufa-documentation branchSeptember 14, 2016 15:08
@den-run-ai
Copy link
ContributorAuthor

I started working on draft release notes. Let's mark all issues to be
resolved for this release.

On Wed, Sep 14, 2016 at 10:08 AM, Benedikt Reinartz <
notifications@github.com> wrote:

Merged#5#5.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#5 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHgZ5VVqMSjOPnkZIrnxV1JrkSuyRiCxks5qqA3ugaJpZM4J7QJD
.

Sign up for freeto 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.

3 participants
@den-run-ai@tonyroberts@filmor

[8]ページ先頭

©2009-2025 Movatter.jp