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

Create cdn_definitions.api module [RHELDST-4862] #18

Closed

Conversation

crungehottman
Copy link
Member

The cdn_definitions.api module allows clients to consume the
data.json or the schema.json files via a convenient API.

This commit also introduces an optional "source" argument to the
data_loads method. The source argument allows the user to specify
the source of the cdn_definitions data.json file. The source could
be a URL or a local path on a directory tree. If a source is not
specified, the location may be overriden via the
CDN_DEFINITIONS_PATH environment variable, which also may either
be in the form of a URL or a local path.

@rbikar
Copy link
Member

rbikar commented Feb 1, 2021

@crungehottman
Can you please review Travis CI? https://travis-ci.org/github/release-engineering/cdn-definitions/builds/756802054
It looks that it fails on deps for py27 and there was an issue when running black formatter.
This also applies for #19

The cdn_definitions.api module allows clients to consume the
data.json or the schema.json files via a convenient API.

This commit also introduces an optional "source" argument to the
data_loads method. The source argument allows the user to specify
the source of the cdn_definitions data.json file. The source could
be a URL or a local path on a directory tree. If a source is not
specified, the location may be overriden via the
CDN_DEFINITIONS_PATH environment variable, which also may either
be in the form of a URL or a local path.
@crungehottman
Copy link
Member Author

@crungehottman
Can you please review Travis CI? https://travis-ci.org/github/release-engineering/cdn-definitions/builds/756802054
It looks that it fails on deps for py27 and there was an issue when running black formatter.
This also applies for #19

PR for deps fix: #20

@rohanpm
Copy link
Member

rohanpm commented Feb 2, 2021

Do you just want to close this since it's included in #19 ? I don't have a problem reviewing them both together.

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

Successfully merging this pull request may close these issues.

None yet

3 participants