Skip to content
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

Implement metrics command (google/clasp#353) #757

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kalmi
Copy link
Contributor

@kalmi kalmi commented Mar 23, 2020

Fixes #353. CI breakage is expected due to scopes change.

Only basic metrics listing is implemented from the data provided by the getMetrics API.

Example:

$ clasp metrics
UTC Dates
2020-03-20  0 users   0 executions  0 failed
2020-03-19  0 users   0 executions  0 failed
2020-03-18  0 users   0 executions  0 failed
2020-03-17  0 users   0 executions  0 failed
2020-03-16  0 users   0 executions  0 failed
2020-03-15  1 user    1 execution   0 failed
2020-03-14  2 users  21 executions  2 failed
  • npm run test succeeds.
  • npm run lint succeeds.
  • Appropriate changes to README are included in PR.

A separate issue could be opened for extending the metrics command with data provided by the processes API. There is a lot of potential in the processes API for extending this command.

@PopGoesTheWza
Copy link
Collaborator

@kalmi can you give a look at the failing test?
TIA

@kalmi
Copy link
Contributor Author

kalmi commented May 26, 2020

I think the token used by the CI needs to be recreated with the new scopes. I think it comes from the CREDS env variable.

@PopGoesTheWza
Copy link
Collaborator

@kalmi sorry for the late reply, I no longer use clasp and rarely check this repo. If this PR is still relevant, can you give a shot at resolving the above conflicts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

clasp metrics
3 participants