PR#1942 Merged Added docstring for is_valid_ssh_key and are_valid_ssh_keys

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

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

  

  

  def is_valid_ssh_key(key):

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

      key = key.strip()

      if not key:

          return None

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

  

  

  def are_valid_ssh_keys(keys):

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

      return all([is_valid_ssh_key(key) is not False

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

  

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

9 months ago

rebased

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

Commit 6a9da69 fixes this pull-request

9 months ago

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

Changes summary
+2 -0
file changed