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

Documentation: rewrite Python docs #13320

Merged
merged 1 commit into from
Mar 5, 2016
Merged

Documentation: rewrite Python docs #13320

merged 1 commit into from
Mar 5, 2016

Conversation

FRidh
Copy link
Member

@FRidh FRidh commented Feb 21, 2016

@FRidh FRidh added 9.needs: reporter feedback This issue needs the person who filed it to respond 2.status: work-in-progress This PR isn't done 8.has: documentation labels Feb 21, 2016
@FRidh FRidh self-assigned this Feb 21, 2016
@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @vcunat, @edolstra and @peti to be potential reviewers

@FRidh FRidh removed the 9.needs: reporter feedback This issue needs the person who filed it to respond label Feb 21, 2016
add a [shebang](https://en.wikipedia.org/wiki/Shebang_(Unix)) to your script
specifying which dependencies Nix shell needs. With the following shebang, you
can use `nix-shell myscript.py` and it will make available all dependencies and
run the script in the `python3` shell.
Copy link
Member

Choose a reason for hiding this comment

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

Maybe a full example could help as reference here:

 #! /usr/bin/env nix-shell
#! nix-shell -i python3 -p python3Packages.numpy

import numpy

print(numpy.__version__)

Copy link
Member Author

Choose a reason for hiding this comment

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

Simple and clear example. Thanks.

@FRidh
Copy link
Member Author

FRidh commented Feb 22, 2016

Just updated this PR. Unfortunately I get the following errors:

manual.xml:7: element para: Relax-NG validity error : Did not expect element para there
manual.xml:3: element section: Relax-NG validity error : Did not expect element section there
manual.xml:3: element section: Relax-NG validity error : Element chapter has extra content: section
manual.xml:4: element info: Relax-NG validity error : Element book has extra content: info
manual.xml fails to validate

@FRidh FRidh added this to the 16.03 milestone Feb 23, 2016
@domenkozar
Copy link
Member

Missed this. That error means you have a tag where it shouldn't be, but it won't tell you where. Good luck :)

@domenkozar
Copy link
Member

I'll look into it, looks amazing on first sight.

@domenkozar domenkozar assigned domenkozar and FRidh and unassigned FRidh and domenkozar Feb 26, 2016
Dependencies such as libraries are automatically installed and should not be
installed explicitly.

The same goes for Python applications and Python packages that are used as
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 a tiny bit confusing. Maybe just drop this sentance? Not really sure what information it adds

@domenkozar
Copy link
Member

Looks amazing, just a few nitpicks :)

@garbas
Copy link
Member

garbas commented Mar 1, 2016

@FRidh: 🍻

@FRidh
Copy link
Member Author

FRidh commented Mar 2, 2016

Thanks for the feedback. I'm fine with removing myEnvFun. I'm a bit busy now but hopefully this weekend I can finish it.

- Rewrite current documentation
- Add introduction/tutorial
- Convert to markdown
FRidh added a commit that referenced this pull request Mar 5, 2016
Documentation: rewrite Python docs
@FRidh FRidh merged commit eddbef1 into NixOS:master Mar 5, 2016
@vcunat
Copy link
Member

vcunat commented Mar 6, 2016

Followup: #13700 (so participants get a notification)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants