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

cmdstan: init at 2.9.0 #11642

Merged
merged 1 commit into from
Dec 12, 2015
Merged

cmdstan: init at 2.9.0 #11642

merged 1 commit into from
Dec 12, 2015

Conversation

davidar
Copy link
Contributor

@davidar davidar commented Dec 12, 2015

CmdStan is the command-line interface to the compiler for the probabilistic programming language Stan.

This is my first nixpkg, so I hope I did it properly :)

@pSub pSub added the 8.has: package (new) This PR adds a new package label Dec 12, 2015
'';

meta = {
description = "CmdStan is the command-line interface to Stan";
Copy link
Member

Choose a reason for hiding this comment

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

description should not contain the package name. cf. http://nixos.org/nixpkgs/manual/.

@davidar
Copy link
Contributor Author

davidar commented Dec 12, 2015

@pSub Thanks for the feedback, is it OK now?

@jagajaga
Copy link
Member

Please squash commits into one.

@davidar
Copy link
Contributor Author

davidar commented Dec 12, 2015

@jagajaga Done.

jagajaga added a commit that referenced this pull request Dec 12, 2015
@jagajaga jagajaga merged commit a7987f9 into NixOS:master Dec 12, 2015
@davidar davidar deleted the cmdstan branch December 12, 2015 13:28
@Janik-Haag Janik-Haag added the 12. first-time contribution This PR is the author's first one; please be gentle! label Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.has: package (new) This PR adds a new package 9.needs: clean-up 12. first-time contribution This PR is the author's first one; please be gentle!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants