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

link-git-protocol: upgrade to latest versions of gitoxide crates #776

Merged
merged 1 commit into from
Aug 27, 2021
Merged

link-git-protocol: upgrade to latest versions of gitoxide crates #776

merged 1 commit into from
Aug 27, 2021

Conversation

Byron
Copy link
Contributor

@Byron Byron commented Aug 27, 2021

Now that the stability guide has landed and a few major refactorings have landed along with the new
git-repository Easy mode I feel safe to kick off Byron/gitoxide#164 with
bumping the gitoxide crates to the latest available version.

Copy link
Contributor

@FintanH FintanH left a comment

Choose a reason for hiding this comment

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

Good to see you here @Byron 😄

I approved the workflow run, and it looks like you need to run cargo fmt. For all CI tests, there's a script called ci/run at the top-level.

Also we follow a DCO, so you'll need to sign your commits. So to fix this commit you can do git commit --amend -s :)

Signed-off-by: Sebastian Thiel <sebastian.thiel@icloud.com>
@Byron
Copy link
Contributor Author

Byron commented Aug 27, 2021

This is the first of hopefully many more PRs to come :).

Also it would have helped to read the contribution guide beforehand 😅, shouldn't happen again.

@FintanH
Copy link
Contributor

FintanH commented Aug 27, 2021

Also it would have helped to read the contribution guide beforehand sweat_smile, shouldn't happen again.

No worries, it happens :)

Copy link
Contributor

@FintanH FintanH left a comment

Choose a reason for hiding this comment

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

lgtm 👍

@kim kim merged commit e6f7361 into radicle-dev:master Aug 27, 2021
@github-pages github-pages bot temporarily deployed to github-pages August 27, 2021 12:52 Inactive
@kim
Copy link
Contributor

kim commented Aug 27, 2021

merci

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.

None yet

3 participants