Skip to content
This repository has been archived by the owner on Oct 9, 2023. It is now read-only.

Fix verification logic #455

Closed
wants to merge 1 commit into from
Closed

Fix verification logic #455

wants to merge 1 commit into from

Conversation

EngHabu
Copy link
Contributor

@EngHabu EngHabu commented Jul 6, 2022

Read then delete this section

- Make sure to use a concise title for the pull-request.

- Use #patch, #minor or #major in the pull-request title to bump the corresponding version. Otherwise, the patch version
will be bumped. More details

TL;DR

Please replace this text with a description of what this PR accomplishes.

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

How did you fix the bug, make the feature etc. Link to any design docs etc

Tracking Issue

Remove the 'fixes' keyword if there will be multiple PRs to fix the linked issue

fixes https://github.com/flyteorg/flyte/issues/

Follow-up issue

NA
OR
https://github.com/flyteorg/flyte/issues/

@EngHabu EngHabu changed the title Tokens Fix verification logic Jul 6, 2022
@codecov
Copy link

codecov bot commented Jul 6, 2022

Codecov Report

Merging #455 (857ecd4) into master (a96084f) will increase coverage by 0.07%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##           master     #455      +/-   ##
==========================================
+ Coverage   61.48%   61.56%   +0.07%     
==========================================
  Files         157      157              
  Lines       11224    11226       +2     
==========================================
+ Hits         6901     6911      +10     
+ Misses       3609     3600       -9     
- Partials      714      715       +1     
Flag Coverage Δ
unittests 60.48% <71.42%> (+0.06%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
auth/authzserver/resource_server.go 55.93% <71.42%> (+5.05%) ⬆️
...implementations/workflow_execution_event_writer.go 80.00% <0.00%> (+40.00%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a96084f...857ecd4. Read the comment docs.

Signed-off-by: Haytham Abuelfutuh <[email protected]>

fix

Signed-off-by: Haytham Abuelfutuh <[email protected]>

cleanup

Signed-off-by: Haytham Abuelfutuh <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant