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

LDA version number should not be hard-wired #25

Open
randykerber opened this issue Jan 15, 2018 · 0 comments
Open

LDA version number should not be hard-wired #25

randykerber opened this issue Jan 15, 2018 · 0 comments

Comments

@randykerber
Copy link
Member

Currently, the LDA version number is hard-wired into the source code, in graphs/linkeddataapigraph.class.php.
When updating this to a new version, like from "2.1" to "2.2", the process is to search for occurrences of the string "2.1" and manually change some of them to "2.2". This is a process prone to error. It's easy to confuse the version number of an app from the version number of some included library.

The app version number should be set in one place as a configuration or global constant.

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

No branches or pull requests

1 participant