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

Make conventional easier to use #20

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

epage
Copy link
Contributor

@epage epage commented Dec 5, 2019

I've started porting my code to conventional. I started with Typed but I needed access to the strs and ran into problems with that, so I thought I'd try Simple. Unfortunately, the lifetimes were off with Simple so that didn't work also.

Ed Page added 3 commits December 5, 2019 09:56
Unlike `Deref` or adding `AsRef<str>`, this allows us to decouple the
lifetimes.
Previously, `Simple` would return values whose lifetimes were tied to
`self` but really all the lifetimes are tied to the original `str` which
is longer than `self`.

This also validates that users get the correct lifetimes from `as_str`
with `Typed`.
@epage
Copy link
Contributor Author

epage commented Feb 27, 2020

@JeanMertz any chance we can get progress on some of these issues? If things are still crazy for you, would you be willing to add me as a maintainer on github and crates.io so I can work to unblock myself?

@JeanMertz
Copy link
Member

Hey @epage, things are less crazy, but still crazy enough to keep me away from open source contributions at the moment, unfortunately.

I've added you to the repo and crates.io as a contributor, thanks for keeping the lights on while I'm gone 👍.

@epage
Copy link
Contributor Author

epage commented Apr 2, 2020

Hey, sorry about this but the invitation expired (had to get setup on 2fa and got distracted from that). Can you re-invite?

@epage
Copy link
Contributor Author

epage commented May 6, 2020

Since I haven't heard back, I've gone ahead and forked, for now, though I'd love to see the two crates merge back together eventually

https://github.com/crate-ci/git-conventional

  • Applied this PR
  • Merged Typed and Simple and moved them off of traits
  • Setup CI and got docs working
  • Other misc changes.

@JeanMertz
Copy link
Member

Sorry @epage, somehow I didn't get a notification about your reply, or I must have accidentally marked it as done.

I've sent another invite, but am glad you found a way forward. Feel free to accept the invitation or keep working on your own fork 👍

I'll get back to more active maintainership soon, I've been using jilu some more recently and so plan on needing some of the improvements you've been making and proposing as well.

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

Successfully merging this pull request may close these issues.

2 participants