Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

*: export clients and related types #77

Merged
merged 1 commit into from
Aug 2, 2019

Conversation

sgotti
Copy link
Member

@sgotti sgotti commented Aug 1, 2019

Export clients and related packages.

The main rule is to not import internal packages from exported packages.

The gateway client and related types are totally decoupled from the gateway
service (not shared types between the client and the server).

Instead the configstore and the runservice client currently share many types
that are now exported (decoupling them will require that a lot of types must be
duplicated and the need of functions to convert between them, this will be done
in future when the APIs will be declared as stable).

@sgotti sgotti force-pushed the export_api_client_types branch from a97790a to 27ce96e Compare August 2, 2019 08:12
Export clients and related packages.

The main rule is to not import internal packages from exported packages.

The gateway client and related types are totally decoupled from the gateway
service (not shared types between the client and the server).

Instead the configstore and the runservice client currently share many types
that are now exported (decoupling them will require that a lot of types must be
duplicated and the need of functions to convert between them, this will be done
in future when the APIs will be declared as stable).
@sgotti sgotti force-pushed the export_api_client_types branch from 27ce96e to c1ff28e Compare August 2, 2019 10:02
@sgotti sgotti merged commit b333b6f into agola-io:master Aug 2, 2019
@sgotti sgotti deleted the export_api_client_types branch February 19, 2020 09:08
tulliobotti64 pushed a commit to tulliobotti64/agola that referenced this pull request Oct 19, 2022
tulliobotti64 pushed a commit to tulliobotti64/agola that referenced this pull request Oct 19, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant