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

Applying pep-8 styling guide rules to src/loader/meta_join.py #3

Closed
wants to merge 1 commit into from
Closed

Applying pep-8 styling guide rules to src/loader/meta_join.py #3

wants to merge 1 commit into from

Conversation

osantana
Copy link

This is a very small patch that only apply pep-8 styling guide rules to the src/loader/meta_join.py.

I'm applying this patch on my forked version to remove error/warning messages when I run pep8.py in my continuous integration environment.

This patch remove some trailing whitespaces too.

ericf referenced this pull request in ericf/yui3 Aug 5, 2011
@davglass
Copy link
Member

This pull can't be merged automatically due to changes in this file since it was submitted (before a few weeks ago we didn't accept pull requests from GitHub so this was not tracked). Closing until it can be merged properly.

@davglass davglass closed this Oct 10, 2011
ericf referenced this pull request in ericf/yui3 Jan 25, 2012
* typo in getRow
* move caption and summary attr => _uiSet* to _UI_ATTRS
* rename {classes} placeholder {className}
* header cells get column class as well
* header cells get yui3-datatable-header class
* data cells get yui3-datatable-cell class
* Core now assigns columns a _id which is name, key, or yuid w/ uniqueness counter
* column class names now yui3-datatable-col-foo (added -col-)
* use toJSON() to get data for the formatters
* replace CSS targeting tags with classes
kloots pushed a commit to kloots/yui3 that referenced this pull request Mar 28, 2012
Convert panel sprites to PNG
@ericf ericf mentioned this pull request Nov 1, 2012
ericf added a commit that referenced this pull request May 14, 2013
@drjayvee drjayvee mentioned this pull request Aug 2, 2013
5 tasks
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.

2 participants