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

Remove the requirment forArtifacts to use a path #8

Open
arcanefoam opened this issue May 17, 2017 · 1 comment
Open

Remove the requirment forArtifacts to use a path #8

arcanefoam opened this issue May 17, 2017 · 1 comment
Labels
bachelor thesis Describes issues which could be used as a basis for a bachelor thesis enhancement

Comments

@arcanefoam
Copy link
Contributor

Currently Artifacts need to use Path instances to point to the files in the local file system that constitute the product "code" base. This makes sense for some specific cases, for example for the ECCO service that provides a "repository" like functionality where the files that constitute a product are kept in a local folder.

In the general sense, ECCO should support files located "anywhere" (e.g. web servers, ftp servers, network drives, etc.). For this I think the best approach is to change the Path to a URI.

If this is defeninetly not the case and things must be kept local, then the generic parameter from Artifacts should be removed and the Path interface "hard codeded".

@meik99
Copy link
Contributor

meik99 commented Jun 15, 2023

agreed, imo the interfaces are too specific to file io and should be refactored into something more generic anyway. The basis of a, e.g., a reader, is, to take an object, convert it to nodes, and return the nodes. It shouldn't matter if the object is a file or something else entirely.

@meik99 meik99 added enhancement bachelor thesis Describes issues which could be used as a basis for a bachelor thesis labels Jun 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bachelor thesis Describes issues which could be used as a basis for a bachelor thesis enhancement
Projects
None yet
Development

No branches or pull requests

2 participants