Skip to content

Various improvements #2

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

Merged
merged 3 commits into from
May 26, 2017
Merged

Various improvements #2

merged 3 commits into from
May 26, 2017

Conversation

whitequark
Copy link
Contributor

Please see individual commits for descriptions.

@japaric
Copy link
Collaborator

japaric commented May 25, 2017

@calebmer friendly ping. Any chance you could review this PR? We have on PR blocked on this one. 🙂

@calebmer
Copy link
Owner

Looks good 👍

@calebmer calebmer merged commit 8bbe6c5 into calebmer:master May 26, 2017
@japaric
Copy link
Collaborator

japaric commented May 26, 2017

@calebmer Thank you! Could you publish a new release with these changes?

@calebmer
Copy link
Owner

calebmer commented Jun 2, 2017

@japaric I’ll get to that tonight. Would you be interested in future maintenance of this crate?

@japaric
Copy link
Collaborator

japaric commented Jun 2, 2017

I’ll get to that tonight.

Thank you!

Would you be interested in future maintenance of this crate?

Sure :-).

@calebmer
Copy link
Owner

calebmer commented Jun 2, 2017

@japaric if you could send me over the links and emails necessary to add you that would be great 😊

@japaric
Copy link
Collaborator

japaric commented Jun 3, 2017

Hmm, so I think you'll have to add me as a collaborator to this repository using
this link:

https://github.com/calebmer/inflections/settings/collaboration

And then add me as an owner of the inflections crate on crates.io using this
command:

$ cd /path/to/inflections

$ cargo owner --add japaric

I think that should do it. (I have never used the cargo owner command before
but that's what the documentation says.)

@calebmer
Copy link
Owner

calebmer commented Jun 3, 2017

@japaric done. Feel free to publish the next release 😊

@japaric
Copy link
Collaborator

japaric commented Jun 3, 2017

@calebmer Thank you! Will do.

japaric pushed a commit to rust-embedded/svd2rust that referenced this pull request Jun 5, 2017
Normalize SVD names to upper case where they were to pascal case

Fixes #76.

Depends on calebmer/inflections#2.
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.

3 participants