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

Avoid highlighting attributes as builtins. #651

Closed
wants to merge 0 commits into from

Conversation

zvezdan
Copy link

@zvezdan zvezdan commented Feb 21, 2016

The need for this was proposed by Carlos Pita.

I had to adapt the implementation to ensure that .1 and similar are
still highlighted or to show ">>> obj.method()" inside a
docstring/doctest in the original color instead of without color.

I've been using this version for months without noticing any issues.
It's ready for merge.

@zvezdan
Copy link
Author

zvezdan commented Mar 5, 2016

Can we get this merged?
It's been rebased from master and passes all the tests.

@brammool
Copy link
Contributor

brammool commented Mar 6, 2016

Zvezdan Petkovic wrote:

Can we get this merged?
It's been rebased from master and passes all the tests.

I prefer getting the new version of the file emailed.

Github doesn't make it easy to just get a raw diff...

EXPERIENCE - experience is a wonderful thing. It enables you to
recognise a mistake when you make it again.

/// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net \
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\ an exciting new programming language -- http://www.Zimbu.org ///
\ help me help AIDS victims -- http://ICCF-Holland.org ///

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.

None yet

2 participants