PR#1942 Merged Added docstring for is_valid_ssh_key and are_valid_ssh_keys

Proposed a year ago by rishika7000
Modified a year 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 :)

a year ago


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

Commit 6a9da69 fixes this pull-request

a year ago

Pull-Request has been merged by

Changes summary
+2 -0
file changed