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

Improve maintainability of this app #41

Closed
rubensworks opened this issue Oct 15, 2020 · 0 comments
Closed

Improve maintainability of this app #41

rubensworks opened this issue Oct 15, 2020 · 0 comments

Comments

@rubensworks
Copy link
Member

Right now, several things are hardcoded in the application, which may make things hard to maintain if some Comunica-related things change.

I would suggest to improve this by looking in these directions:

  • Create separate template files for mediator and actor configs (also package.json), so we can easily change these should Comunica's config structure change.
  • Move URLs to a config file (such as https://linkedsoftwaredependencies.org/)
  • Dynamically determine latest package versions (in particular, @comunica/actor-init-sparql is now fixed at 1.17.0, but this changes quite often, also components.js) (Note that the linkedsoftwaredependencies also include this version (only the major part, so ^1.17.0 -> ^1.0.0)
  • ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants