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

Rename repo to denso if possible #13

Closed
130s opened this issue Dec 12, 2013 · 4 comments

Comments

Projects
None yet
2 participants
@130s
Copy link
Contributor

commented Dec 12, 2013

Discussed here.

@ghost ghost assigned 130s Dec 12, 2013

@130s

This comment has been minimized.

Copy link
Contributor Author

commented Dec 16, 2013

This case is settled; we'll rename our packages.

For descriptions, we'll also pay attention to #14.

@wyasuda fyi.

@gavanderhoorn

This comment has been minimized.

Copy link

commented Dec 16, 2013

Just curious: are you going to go with an all capitals package name? Seeing as the ROS naming guidelines explicitly allow only the use of lowercase alphanumerics (and underscores)?

@130s

This comment has been minimized.

Copy link
Contributor Author

commented Dec 20, 2013

@gavanderhoorn Nope, as I said I'll apply all-capital rule to the package description only (I'm aware of ROS package naming convention and those where package name start with capital shed so many annoying warnings).

@130s

This comment has been minimized.

Copy link
Contributor Author

commented Jan 3, 2014

Fix via 817a7a8. #14 is done as well.

@130s 130s closed this Jan 3, 2014

130s pushed a commit to 130s/rosdistro that referenced this issue Jan 3, 2014

Isao Isaac Saito
Rename repository densowave to denso. 0.2.7-0 in 'hydro/release.yaml'…
… (ref. start-jsk/denso#13)

Re-ordering denso's sub packages due to package name change.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.