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

- gphoto2-2.4.14, libgphoto2-2.4.14 #165

Merged
merged 1 commit into from
Oct 21, 2012
Merged

- gphoto2-2.4.14, libgphoto2-2.4.14 #165

merged 1 commit into from
Oct 21, 2012

Conversation

jcumming
Copy link
Contributor

No description provided.

@aszlig
Copy link
Member

aszlig commented Oct 21, 2012

Git-nazi speaking: Why are you prepending every commit summary with a dash?
Other than that, it looks good to me (except that the whitespace-nazi in me thinks that the blank lines in longDescription are unnecessary as well).

@aszlig aszlig merged commit 1db3488 into NixOS:master Oct 21, 2012
@jcumming
Copy link
Contributor Author

Prepending comments with dashes is a habit carried over from version control systems that have log outputs that are hard to parse without some visual cue to WTF the commit message is. I'll drop them in the interest of uniform style.

Profpatsch pushed a commit that referenced this pull request Jun 20, 2020
pyrox0 added a commit to pyrox0/nixpkgs that referenced this pull request Jul 9, 2024
pyrox0 added a commit to pyrox0/nixpkgs that referenced this pull request Jul 9, 2024
pyrox0 added a commit to pyrox0/nixpkgs that referenced this pull request Jul 13, 2024
nim65s added a commit to nim65s/nixpkgs that referenced this pull request Aug 13, 2024
To try to see why why nixpkgs-review does not list casadi on aarch64-darwin

We still can see
`165/165 Test NixOS#165: example-py-casadi-quadrotor-ocp ............  Passed`
in `nix log .#python3Packages.pinocchio` on aarch64-darwin, so
everything looks fine anyways.
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.

2 participants