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

Remove lockfile #46

Merged
merged 2 commits into from
Sep 11, 2022
Merged

Remove lockfile #46

merged 2 commits into from
Sep 11, 2022

Conversation

matthiasbeyer
Copy link
Member

No description provided.

As this is a library crate, we do not want the Cargo.lock in the
repository.

Signed-off-by: Matthias Beyer <[email protected]>
Signed-off-by: Matthias Beyer <[email protected]>
@matthiasbeyer
Copy link
Member Author

@philippkeller it seems that reviews are required. Can you please disable them? This setting can be managed automatically by bors, if you want to keep this setting (I can file a PR for enabling this with bors)!

@philippkeller
Copy link
Collaborator

@matthiasbeyer sorry, didn't see this before. I don't see where I can disable this?
image
image

@matthiasbeyer
Copy link
Member Author

matthiasbeyer commented Sep 11, 2022 via email

@matthiasbeyer
Copy link
Member Author

bors single on
bors merge

@bors
Copy link
Contributor

bors bot commented Sep 11, 2022

Build succeeded:

@bors bors bot merged commit 089150f into rust-cli:master Sep 11, 2022
@matthiasbeyer matthiasbeyer deleted the remove-lockfile branch September 11, 2022 18:56
@matthiasbeyer
Copy link
Member Author

@philippkeller Just to make sure you do not miss this: This PR was merged, but the setting is still not disabled.

@philippkeller
Copy link
Collaborator

@matthiasbeyer I just changed the setting now, can you check again? Thanks so much for everything. This project has never been so active ever 🎉

@matthiasbeyer
Copy link
Member Author

@philippkeller yes looks alright now. Thanks a lot!

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