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.

Sign up for GitHub

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

Jump to bottom

Refactor configuration parsing #114

Open
mrogaski opened this issue Aug 29, 2017 · 0 comments
Open

Refactor configuration parsing #114

mrogaski opened this issue Aug 29, 2017 · 0 comments
Assignees
@mrogaski
Labels
refactor

Comments

@mrogaski
Copy link
Member

Move the configuration parser code to a separate class.
@mrogaski mrogaski added this to the Accepted milestone Aug 29, 2017
@mrogaski mrogaski self-assigned this Aug 29, 2017
mrogaski added a commit that referenced this issue Aug 29, 2017
@mrogaski
Added versio 1 configuration method. (#114)
f88fbe8
@mrogaski mrogaski changed the title Rafactor configuration parsing Refactor configuration parsing Sep 1, 2017
mrogaski added a commit that referenced this issue Dec 30, 2017
@mrogaski
Added factory method and parser subclasses. (#114)
53a9dc2
mrogaski added a commit that referenced this issue Dec 30, 2017
@mrogaski
Cleaned up class inheritance implementation. (#114)
f01022c
mrogaski added a commit that referenced this issue Dec 31, 2017
@mrogaski
Reordered the Parser.lua file. (#114)
63109a1
mrogaski added a commit that referenced this issue Jan 1, 2018
@mrogaski
Renamed GwConfig to GwState. (#114)
29ab8c7 
In order to restructure the configuration parsing,
all of the running state will be under gw.state and the
configuration will consist only of parsed configuration data
from the environment.
mrogaski added a commit that referenced this issue Jan 2, 2018
@mrogaski
Added new GwConfig and GwConfigBuilder with basic unit tests. (#114)
e2a329f
mrogaski added a commit that referenced this issue Jan 2, 2018
@mrogaski
Added config unit tests to Travis. (#114)
d36cecb
mrogaski added a commit that referenced this issue Jan 3, 2018
@mrogaski
Added unit tests. (#114)
a21b5c4
mrogaski added a commit that referenced this issue Jan 8, 2018
@mrogaski
Restructured some parser helpers. (#114)
b92121d
mrogaski added a commit that referenced this issue Jan 8, 2018
@mrogaski
Fixed unit test. (#114)
d728fa7
@mrogaski mrogaski added refactor and removed active labels Jul 19, 2018
@mrogaski mrogaski removed this from the Accepted milestone Jan 5, 2019
@mrogaski mrogaski pinned this issue Jan 5, 2019
@mrogaski mrogaski unpinned this issue Jan 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mrogaski mrogaski

Labels
refactor
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mrogaski