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

Fix Issue #26 #27

Closed
wants to merge 6 commits into from
Closed

Fix Issue #26 #27

wants to merge 6 commits into from

Conversation

rvesse
Copy link

@rvesse rvesse commented Nov 13, 2013

Modifies Parser so that the allowedValues for an option are honoured and
an error thrown on illegal values

Includes unit test which verifies the improve behaviour

Modifies Parser so that the allowedValues for an option are honoured and
an error thrown on illegal values
@rvesse
Copy link
Author

rvesse commented Nov 15, 2013

@dain Copyright header should now be Apache 2 like the rest of airline

@rvesse
Copy link
Author

rvesse commented Dec 10, 2014

I have brought my fork back up to date with master and this fix can now be merged again

@rvesse
Copy link
Author

rvesse commented Jan 12, 2015

I've now forked my own variant of airline with this and various other changes I wanted so I am closing this pull request since this is no longer a self-contained feature in my fork

@rvesse rvesse closed this Jan 12, 2015
@cread
Copy link

cread commented Nov 1, 2018

Why did this never get fixed? It's causing me problems right now...

@rvesse
Copy link
Author

rvesse commented Nov 2, 2018

@cread My fork at http://github.com/rvesse/airline has this and many other fixes that have never been accepted here

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

Successfully merging this pull request may close these issues.

3 participants