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

Fix config.IsExists for corner case #70

Closed
jeevatkm opened this issue Jun 14, 2017 · 1 comment
Closed

Fix config.IsExists for corner case #70

jeevatkm opened this issue Jun 14, 2017 · 1 comment
Assignees
@jeevatkm
Labels
bug lib-config Configuration
Projects
aah Roadmap
Milestone
v0.7 Milestone

Comments

@jeevatkm
Copy link
Member

Currently config.IsExists returns false for corner cases, It good to use config.Get method to verify key existence.
@jeevatkm jeevatkm added bug lib-config Configuration labels Jun 14, 2017
@jeevatkm jeevatkm added this to the v0.7 Milestone milestone Jun 14, 2017
@jeevatkm jeevatkm self-assigned this Jun 14, 2017
jeevatkm added a commit to go-aah/config that referenced this issue Jun 14, 2017
@jeevatkm
version bump to v0.4.2 and go-aah/aah#70 isexists fix
854b187
@jeevatkm
Copy link
Member Author

Fixed :)

@jeevatkm jeevatkm added this to In Progress in aah Roadmap Jun 14, 2017
@jeevatkm jeevatkm moved this from In Progress to Done in aah Roadmap Jun 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jeevatkm jeevatkm

Labels
bug lib-config Configuration
Projects
aah Roadmap
  
Released to Audience
Milestone
v0.7 Milestone
Development

No branches or pull requests
1 participant
@jeevatkm