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

add back David's name into the authors list #75

Merged
merged 2 commits into from
Jun 16, 2022
Merged

Conversation

rahulanand16nov
Copy link
Contributor

I think @davidor 's name was removed mistakenly. Though he's not going to contribute still he's still an author, right? :)

@@ -1,7 +1,7 @@
[package]
name = "limitador-server"
version = "0.5.1"
authors = ["Eguzki Astiz Lezaun <[email protected]>"]
authors = ["Eguzki Astiz Lezaun <[email protected]>", "David Ortiz <[email protected]>"]
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
authors = ["Eguzki Astiz Lezaun <[email protected]>", "David Ortiz <[email protected]>"]
authors = ["Rahul Anand <[email protected]>", "Eguzki Astiz Lezaun <[email protected]>", "David Ortiz <[email protected]>"]

@rahulanand16nov
Copy link
Contributor Author

@davidor I think current maintainers should be first on the list, is it okay with you? Can we get ownership of https://crates.io/crates/limitador so we can push the new release?

@rahulanand16nov
Copy link
Contributor Author

Let's wait for @davidor's reply before merging this.

@eguzki
Copy link
Contributor

eguzki commented Jun 16, 2022

@alexsnaps should be also added

@rahulanand16nov rahulanand16nov merged commit 8bd1bda into main Jun 16, 2022
@rahulanand16nov rahulanand16nov deleted the fix-authors branch June 16, 2022 08:50
@eguzki
Copy link
Contributor

eguzki commented Jun 16, 2022

@rahulanand16nov you did not accept my suggested change

@rahulanand16nov
Copy link
Contributor Author

@eguzki I haven't contributed much yet but will add once I have contributed in a bit more significant way 😅

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