-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix/add token #148
Fix/add token #148
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## next #148 +/- ##
=======================================
Coverage 97.73% 97.73%
=======================================
Files 23 23
Lines 1983 1983
Branches 284 284
=======================================
Hits 1938 1938
Misses 45 45 ☔ View full report in Codecov by Sentry. |
@cre8 I think it includes other parts besides the token modification. |
reverted the changes, now its ready |
Signed-off-by: Mirko Mollik <[email protected]>
Signed-off-by: Mirko Mollik <[email protected]>
Signed-off-by: Mirko Mollik <[email protected]>
Signed-off-by: Mirko Mollik <[email protected]>
This reverts commit 391a365. Signed-off-by: Mirko Mollik <[email protected]>
Signed-off-by: Mirko Mollik <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good :)
@cre8 should this pr merge to main? |
I first wanted to push it to next and then to main |
Signed-off-by: Mirko Mollik <[email protected]> Co-authored-by: github-actions <[email protected]> Signed-off-by: Mirko Mollik <[email protected]>
Signed-off-by: Mirko Mollik <[email protected]> Co-authored-by: github-actions <[email protected]> Signed-off-by: Mirko Mollik <[email protected]>
No description provided.