almalinux/cas_wrapper
13
0
Fork 2
Code Issues Pull Requests Releases Activity

Added notarize_no_exc method to return success state instead of raising the exception #5

Merged
Korulag merged 2 commits from albs-637 into master 2022-09-14 20:02:22 +00:00
Conversation 0 Commits 2 Files Changed 2 +25 -1
Korulag commented 2022-09-14 15:59:22 +00:00
Member
Copy Link
No description provided.
Korulag added 1 commit 2022-09-14 15:59:23 +00:00
soksanichenko reviewed 2022-09-14 16:17:22 +00:00
cas_wrapper.py
@ -79,6 +79,21 @@ class CasWrapper:
result_of_execution = command()
return json.loads(result_of_execution)['hash']
def notarize_no_exc(
soksanichenko commented 2022-09-14 16:17:18 +00:00
Owner
Copy Link

Please, add docstring, or expand name of the method. It's unclear

Please, add docstring, or expand name of the method. It's unclear
soksanichenko marked this conversation as resolved
Korulag added 1 commit 2022-09-14 16:28:54 +00:00
soksanichenko approved these changes 2022-09-14 16:37:04 +00:00
mxpthv approved these changes 2022-09-14 19:11:37 +00:00
jhernandez approved these changes 2022-09-14 19:25:52 +00:00
Korulag merged commit 211d4521c0 into master 2022-09-14 20:02:22 +00:00
Korulag deleted branch albs-637 2022-09-14 20:02:28 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
soksanichenko
mxpthv
jhernandez
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: almalinux/cas_wrapper#5
No description provided.
Delete Branch "albs-637"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?