Skip to content

General cleanup#282

Merged
maxkorp merged 7 commits into
masterfrom
general-cleanup
Nov 21, 2014
Merged

General cleanup#282
maxkorp merged 7 commits into
masterfrom
general-cleanup

Conversation

@maxkorp
Copy link
Copy Markdown
Collaborator

@maxkorp maxkorp commented Nov 21, 2014

A few things here.

  • A few tweaks to generation to reduce the number of overrides
    • fromsomething/bysomething will now evaluate like from_something/by_something
    • any method ending in _str (or by proxy, fromstr etc) will be evaluated like _string
    • (coming soon) enums can take a replacement string in the descriptor to reduce duplicate overrides
  • Refs renamed to Reference
  • Fixed a few weird test issues with promises
  • Removed a bunch of client manual enums

@maxkorp
Copy link
Copy Markdown
Collaborator Author

maxkorp commented Nov 21, 2014

Lastly, this removes some version compat stuff like weird renames. Should be ready to go, soon as travis stops losing its mind.

@maxkorp maxkorp changed the title [WIP] General cleanup General cleanup Nov 21, 2014
maxkorp added a commit that referenced this pull request Nov 21, 2014
@maxkorp maxkorp merged commit 4462aaf into master Nov 21, 2014
@maxkorp maxkorp deleted the general-cleanup branch November 21, 2014 03:19
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.

1 participant