PR#1942 Merged Added docstring for is_valid_ssh_key and are_valid_ssh_keys

Proposed 5 months ago by rishika7000
Modified 5 months ago
From forks/rishika7000/pagure donedone  into pagure master
file changed

 1 @@ -192,6 +192,7 @@ 

 2   

 3   

 4   def is_valid_ssh_key(key):

 5 + """ Validates the ssh key using ssh-keygen. """

 6       key = key.strip()

 7       if not key:

 8           return None

 9 @@ -210,6 +211,7 @@

10   

11   

12   def are_valid_ssh_keys(keys):

13 + """ Checks if all the ssh keys are valid or not. """

14       return all([is_valid_ssh_key(key) is not False

15                   for key in keys.split('\n')])

16   

Looks fine to me, let's rebase and merge :)

5 months ago

rebased

To save you the rebase again, I'm going to merge this one manually, thanks for the PR! :)

Commit 6a9da694 fixes this pull-request

5 months ago

Pull-Request has been merged by rishika7000 @gmail.com

Changes summary
+2 -0
file changed