-
Notifications
You must be signed in to change notification settings - Fork 504
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
🌱 Add url for openfga #3337
🌱 Add url for openfga #3337
Conversation
Signed-off-by: Andres Aguiar <andres.aguiar@gmail.com>
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.
You need a comma, as this is a CSV file with 2 columns.
make add-projects
should help confirm once you did it correctly.
Signed-off-by: Andres Aguiar <andres.aguiar@gmail.com>
@spencerschrock thanks for the quick review! I apologize, just fixed it and tested with |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #3337 +/- ##
===========================================
- Coverage 73.26% 61.54% -11.72%
===========================================
Files 173 173
Lines 12717 12717
===========================================
- Hits 9317 7827 -1490
- Misses 2877 4463 +1586
+ Partials 523 427 -96 |
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.
Thanks
* Add https://github/openfga/openfga repository Signed-off-by: Andres Aguiar <andres.aguiar@gmail.com> * Added missing comma Signed-off-by: Andres Aguiar <andres.aguiar@gmail.com> --------- Signed-off-by: Andres Aguiar <andres.aguiar@gmail.com> Signed-off-by: Allen Shearin <allen.p.shearin@gmail.com>
What kind of change does this PR introduce?
Add url for https://github.com/openfga/openfga repository.
What is the current behavior?
What is the new behavior (if this is a feature change)?**
Which issue(s) this PR fixes
NONE
Special notes for your reviewer
Does this PR introduce a user-facing change?