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

Add dfn as anchor for interfaces, dictionaries, enums, ... #989

Merged
merged 1 commit into from Jan 12, 2017
Merged

Add dfn as anchor for interfaces, dictionaries, enums, ... #989

merged 1 commit into from Jan 12, 2017

Conversation

dontcallmedom
Copy link
Member

Fix (most of) respec warnings

@dontcallmedom dontcallmedom mentioned this pull request Jan 10, 2017
Closed
@alvestrand alvestrand merged commit ebd84fb into master Jan 12, 2017
@alvestrand
Copy link
Contributor

@dontcallmedom master is broken after merge :-(

@dontcallmedom
Copy link
Member Author

@alvestrand that was broken before I think

@alvestrand
Copy link
Contributor

@dontcallmedom maybe... but you find these things faster than me....

@dontcallmedom
Copy link
Member Author

looking at it quickly, this seems a respec bug - will look more into it

@dontcallmedom
Copy link
Member Author

so I have a work around in #992 - I think ReSpec doesn't expect enums to have upper case values (which indeed goes against what's recommended); I'm working on a patch for respec, but if #992 is acceptable, it's very likely to be much faster to fix Travis that way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@dontcallmedom @alvestrand