-
Notifications
You must be signed in to change notification settings - Fork 20
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 slow version resolution #54
Conversation
we were making 100 calls to apibuilder and 100 subprocesses of `git describe`
bin/apibuilder
Outdated
end | ||
end | ||
found | ||
versions = client.versions.get_by_application_key(org, application, limit: 100).map { |v| v.version } |
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.
the api says:
The latest version is the first result returned.
https://app.apibuilder.io/apicollective/apibuilder-api/latest
so I think we can just limit 1
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.
we are actually hitting this endpoint: https://app.apibuilder.io/apicollective/apibuilder-api/latest#resource-version
and I don't think we can do limit=1 because what if there are new versions in apibuilder that you don't have locally?
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.
ack
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.
let's merge - ryan can review post merge
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.
lgtm. What's happening here? Just lots of change to the codebase with no changes to the API document?
we were making 100 calls to apibuilder and 100 subprocesses of
git describe