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

Outdated references to graph.properties #2725

Closed
optionsome opened this issue Feb 19, 2019 · 1 comment
Closed

Outdated references to graph.properties #2725

optionsome opened this issue Feb 19, 2019 · 1 comment

Comments

@optionsome
Copy link
Member

While browsing through changes made in #2516 I noticed there are still references to graph.properties file, which as far as I know, can't be used anymore in the current version of OTP. I think those comments should be updated as they are confusing unless graph.properties file can somehow still be used.

Expected behavior

There should be no references to graph.properties anymore.

Observed behavior

There are comments that still refer to graph.properties file which is not being used anymore as it was split into router-config.json and builder-config.json

@t2gran t2gran added X OTP1 ~ Not in use any more ~ Fix or backport to the 1.x version of OTP OTP2 labels May 3, 2019
@abyrd abyrd added this to the 2.0 milestone Oct 13, 2020
@abyrd abyrd removed the OTP2 label Oct 13, 2020
@t2gran t2gran added Documentation and removed X OTP1 ~ Not in use any more ~ Fix or backport to the 1.x version of OTP labels Nov 13, 2020
@t2gran t2gran self-assigned this Nov 13, 2020
@t2gran
Copy link
Member

t2gran commented Nov 17, 2020

This is fixed.

@t2gran t2gran closed this as completed Nov 17, 2020
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

3 participants