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

Remove from __future__ import absolute_import. #6997

Closed
wants to merge 1 commit into from

Conversation

rht
Copy link
Collaborator

@rht rht commented Oct 14, 2017

Except in:

  • docs/writing-bots-guide.md, because bots are supposed to be Python 2
    compatible
  • puppet/zulip_ops/files/zulip-ec2-configure-interfaces, because this
    script is still on python2.7
  • tools/lint
  • tools/linter_lib

@timabbott
Copy link
Sponsor Member

@rht as I've mentioned before, I don't want to remove any this stuff from tools/linter_lib and tools/lint, since those are likely to be split out as a separate open source project with Python 2 support.

Except in:
- docs/writing-bots-guide.md, because bots are supposed to be Python 2
  compatible
- puppet/zulip_ops/files/zulip-ec2-configure-interfaces, because this
  script is still on python2.7
- tools/lint
- tools/linter_lib

For the latter two, because they might be yanked away to a separate repo
for general use with other FLOSS projects.
@rht
Copy link
Collaborator Author

rht commented Oct 17, 2017

Re-added to tools/linter_lib and tools/lint, added note to the commit message.

@rht
Copy link
Collaborator Author

rht commented Oct 18, 2017

@timabbott

@timabbott
Copy link
Sponsor Member

Merged, after also excluding the changes to lister.py, which probably belongs in linter_lib.

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

3 participants