Skip to content
This repository has been archived by the owner on Jun 16, 2021. It is now read-only.

Add ServiceNames() to APIProject #462

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

Conversation

yudai
Copy link
Contributor

@yudai yudai commented Apr 27, 2017

Issue Observed

project.APIProject doesn't provide a method to get the service names
define in a project.

You cannot get service names when you use GetSreviceConfig() and other
methods of the interface.

Proposed Resolution

We could add a method GetServiceConfigs() which returns the
*ServiceConfigs owned by the concrete struct (project.Project).
However, since the strcut already has ServiceConfigs property, we cannot add a new method with the same name.

In this commit, we add a new method just returns the list of service
names instead of returning the *ServiceConfigs itself.

Issue Observed
--------------

project.APIProject doesn't provide a method to get the service names
define in a project.

You cannot get service names when you use GetSreviceConfig() and other
methods of the interface.

Proposed Resolution
-------------------

We could add a method GetServiceConfigs() which returns the
*ServiceConfigs owned by the concrete struct (project.Project).
However, since the strcut already has ServiceConfigs property, we cannot add a new method with the same name.

In this commit, we add a new method just returns the list of service
names instead of returning the *ServiceConfigs itself.

Signed-off-by: Iwasaki Yudai <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant