Skip to content
This repository has been archived by the owner on Nov 8, 2024. It is now read-only.

Report not allowed characters in identifiers #650

Open
abtris opened this issue Oct 15, 2013 · 4 comments
Open

Report not allowed characters in identifiers #650

abtris opened this issue Oct 15, 2013 · 4 comments

Comments

@abtris
Copy link
Contributor

abtris commented Oct 15, 2013

@zdne opened apiaryio/snowcrash#41

Currently only some characters are allowed in API Blueprint identifiers.

If illegal character is used the whole section is silently ignored.

Following blueprint should at least produce a warning about illegal identifier (character ')

# Group Parcel 
## Parcel's sticker [/parcels/{parcel_id}/stickers.json]
@abtris
Copy link
Contributor Author

abtris commented Jan 31, 2014

@zdne commented

Also related: apiaryio/snowcrash#70

@abtris
Copy link
Contributor Author

abtris commented Feb 7, 2014

@zdne commented

With apiaryio/snowcrash#70 resolved this only mean that unexpected characters in identifiers are [, ], ( and ).

E.g:

Group Parc ] el

@abtris
Copy link
Contributor Author

abtris commented Sep 24, 2014

@pksunkara commented

What is needed to be done for this to be closed?

@abtris
Copy link
Contributor Author

abtris commented Sep 26, 2014

@zdne commented

Currently a group like this is silently ignored (and treated as a description):

## Group ]A
Hello

To fix this issue the parser should warn that while keyword was find the identifier is not valid. Related issues are apiaryio/snowcrash#31 and apiaryio/snowcrash#65 and should be probably solved in one go.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant