-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Add InstallationsCount to App #2765
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
Conversation
dbc9fff
to
94c4092
Compare
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.
Thank you, @chmouel .
Please run go generate ./...
at the top of the repo and push the results to this PR.
See CONTRIBUTING.md for more details.
2b0710a
to
876a434
Compare
Please see the other change I asked for above and also please make sure all unit tests pass before pushing (not force-pushing since we always squash-and-merge in this repo) changes to the PR. |
@gmlewis apologies for the noise, i kinda messed my pr! |
I am not really sure why it fails on CI as it works locally
is it because i am using go 1.20 ? |
Signed-off-by: Chmouel Boudjnah <chmouel@chmouel.com>
876a434
to
9d5f093
Compare
ah it works better with |
Codecov Report
@@ Coverage Diff @@
## master #2765 +/- ##
=======================================
Coverage 98.05% 98.05%
=======================================
Files 132 132
Lines 11638 11638
=======================================
Hits 11412 11412
Misses 154 154
Partials 72 72
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Feel free to clarify if you think that would be helpful. |
I think it was not just my day today, and just was too quick without thinking much, apologies again for the noise, will be careful next time when contributing. (I missed a dot to |
No worries! Glad you got it all worked out. 😁 |
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.
Thank you, @chmouel !
LGTM.
Merging.
It's not in the official github documentation yet but exposed to the API.