Skip to content

Commit

Permalink
ruff: Collapse short multi-line import statements.
Browse files Browse the repository at this point in the history
isort did this by default, though it’s unclear whether that was
intended; see astral-sh/ruff#4153.

Signed-off-by: Anders Kaseorg <[email protected]>
  • Loading branch information
andersk committed Aug 3, 2023
1 parent bbebf94 commit e0ba641
Show file tree
Hide file tree
Showing 14 changed files with 18 additions and 82 deletions.
1 change: 1 addition & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -178,3 +178,4 @@ extend-function-names = ["gettext_lazy"]

[tool.ruff.isort]
known-third-party = ["zulip"]
split-on-trailing-comma = false
5 changes: 1 addition & 4 deletions zerver/actions/message_send.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,10 +69,7 @@
from zerver.lib.streams import access_stream_for_send_message, ensure_stream
from zerver.lib.string_validation import check_stream_name
from zerver.lib.timestamp import timestamp_to_datetime
from zerver.lib.topic import (
filter_by_exact_message_topic,
participants_for_topic,
)
from zerver.lib.topic import filter_by_exact_message_topic, participants_for_topic
from zerver.lib.url_preview.types import UrlEmbedData
from zerver.lib.user_message import UserMessageLite, bulk_insert_ums
from zerver.lib.validator import check_widget_content
Expand Down
8 changes: 1 addition & 7 deletions zerver/lib/cache_helpers.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,13 +23,7 @@
from zerver.lib.safe_session_cached_db import SessionStore
from zerver.lib.sessions import session_engine
from zerver.lib.users import get_all_api_keys
from zerver.models import (
Client,
Huddle,
UserProfile,
get_client_cache_key,
huddle_hash_cache_key,
)
from zerver.models import Client, Huddle, UserProfile, get_client_cache_key, huddle_hash_cache_key


def user_cache_items(
Expand Down
4 changes: 1 addition & 3 deletions zerver/lib/events.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,7 @@
from django.utils.translation import gettext as _

from version import API_FEATURE_LEVEL, ZULIP_MERGE_BASE, ZULIP_VERSION
from zerver.actions.default_streams import (
default_stream_groups_to_dicts_sorted,
)
from zerver.actions.default_streams import default_stream_groups_to_dicts_sorted
from zerver.actions.users import get_owned_bot_dicts
from zerver.lib import emoji
from zerver.lib.alert_words import user_alert_words
Expand Down
6 changes: 1 addition & 5 deletions zerver/lib/users.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,7 @@
from zulip_bots.custom_exceptions import ConfigValidationError

from zerver.lib.avatar import avatar_url, get_avatar_field
from zerver.lib.cache import (
cache_with_key,
get_cross_realm_dicts_key,
realm_user_dict_fields,
)
from zerver.lib.cache import cache_with_key, get_cross_realm_dicts_key, realm_user_dict_fields
from zerver.lib.exceptions import (
JsonableError,
OrganizationAdministratorRequiredError,
Expand Down
6 changes: 1 addition & 5 deletions zerver/tests/test_invite.py
Original file line number Diff line number Diff line change
Expand Up @@ -43,11 +43,7 @@
from zerver.context_processors import common_context
from zerver.lib.create_user import create_user
from zerver.lib.default_streams import get_default_streams_for_realm_as_dicts
from zerver.lib.send_email import (
FromAddress,
deliver_scheduled_emails,
send_future_email,
)
from zerver.lib.send_email import FromAddress, deliver_scheduled_emails, send_future_email
from zerver.lib.streams import ensure_stream
from zerver.lib.test_classes import ZulipTestCase
from zerver.lib.test_helpers import find_key_by_email
Expand Down
16 changes: 3 additions & 13 deletions zerver/tests/test_signup.py
Original file line number Diff line number Diff line change
Expand Up @@ -18,16 +18,10 @@
from django.utils.translation import gettext as _

from confirmation import settings as confirmation_settings
from confirmation.models import (
Confirmation,
one_click_unsubscribe_link,
)
from confirmation.models import Confirmation, one_click_unsubscribe_link
from zerver.actions.create_realm import do_change_realm_subdomain, do_create_realm
from zerver.actions.create_user import add_new_user_history
from zerver.actions.default_streams import (
do_add_default_stream,
do_create_default_stream_group,
)
from zerver.actions.default_streams import do_add_default_stream, do_create_default_stream_group
from zerver.actions.realm_settings import (
do_deactivate_realm,
do_set_realm_authentication_methods,
Expand All @@ -50,11 +44,7 @@
xor_hex_strings,
)
from zerver.lib.name_restrictions import is_disposable_domain
from zerver.lib.send_email import (
EmailNotDeliveredError,
FromAddress,
send_future_email,
)
from zerver.lib.send_email import EmailNotDeliveredError, FromAddress, send_future_email
from zerver.lib.stream_subscription import get_stream_subscriptions_for_user
from zerver.lib.streams import create_stream_if_needed
from zerver.lib.subdomains import is_root_domain_available
Expand Down
18 changes: 3 additions & 15 deletions zerver/tests/test_upload_local.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,7 @@
import zerver.lib.upload
from zerver.lib.avatar_hash import user_avatar_path
from zerver.lib.test_classes import UploadSerializeMixin, ZulipTestCase
from zerver.lib.test_helpers import (
get_test_image_file,
read_test_image_file,
)
from zerver.lib.test_helpers import get_test_image_file, read_test_image_file
from zerver.lib.upload import (
all_message_attachments,
delete_export_tarball,
Expand All @@ -24,18 +21,9 @@
upload_export_tarball,
upload_message_attachment,
)
from zerver.lib.upload.base import (
DEFAULT_EMOJI_SIZE,
MEDIUM_AVATAR_SIZE,
resize_avatar,
)
from zerver.lib.upload.base import DEFAULT_EMOJI_SIZE, MEDIUM_AVATAR_SIZE, resize_avatar
from zerver.lib.upload.local import write_local_file
from zerver.models import (
Attachment,
RealmEmoji,
get_realm,
get_system_bot,
)
from zerver.models import Attachment, RealmEmoji, get_realm, get_system_bot


class LocalStorageTest(UploadSerializeMixin, ZulipTestCase):
Expand Down
8 changes: 1 addition & 7 deletions zerver/tests/test_upload_s3.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,13 +36,7 @@
resize_avatar,
)
from zerver.lib.upload.s3 import S3UploadBackend
from zerver.models import (
Attachment,
RealmEmoji,
UserProfile,
get_realm,
get_system_bot,
)
from zerver.models import Attachment, RealmEmoji, UserProfile, get_realm, get_system_bot


class S3Test(ZulipTestCase):
Expand Down
5 changes: 1 addition & 4 deletions zerver/tests/test_user_topics.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,7 @@
from zerver.actions.user_topics import do_set_user_topic_visibility_policy
from zerver.lib.stream_topic import StreamTopicTarget
from zerver.lib.test_classes import ZulipTestCase
from zerver.lib.user_topics import (
get_topic_mutes,
topic_has_visibility_policy,
)
from zerver.lib.user_topics import get_topic_mutes, topic_has_visibility_policy
from zerver.models import UserProfile, UserTopic, get_stream


Expand Down
8 changes: 1 addition & 7 deletions zerver/views/message_send.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,13 +22,7 @@
from zerver.lib.validator import check_string_in, to_float
from zerver.lib.zcommand import process_zcommands
from zerver.lib.zephyr import compute_mit_user_fullname
from zerver.models import (
Client,
Message,
RealmDomain,
UserProfile,
get_user_including_cross_realm,
)
from zerver.models import Client, Message, RealmDomain, UserProfile, get_user_including_cross_realm


class InvalidMirrorInputError(Exception):
Expand Down
6 changes: 1 addition & 5 deletions zerver/views/report.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,7 @@

from zerver.lib.request import REQ, has_request_variables
from zerver.lib.response import json_success
from zerver.lib.validator import (
WildValue,
check_string,
to_wild_value,
)
from zerver.lib.validator import WildValue, check_string, to_wild_value


@csrf_exempt
Expand Down
5 changes: 1 addition & 4 deletions zerver/views/sentry.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,7 @@

from zerver.lib.exceptions import JsonableError
from zerver.lib.outgoing_http import OutgoingSession
from zerver.lib.validator import (
check_url,
to_wild_value,
)
from zerver.lib.validator import check_url, to_wild_value


class SentryTunnelSession(OutgoingSession):
Expand Down
4 changes: 1 addition & 3 deletions zproject/urls.py
Original file line number Diff line number Diff line change
Expand Up @@ -130,9 +130,7 @@
realm_register,
signup_send_confirm,
)
from zerver.views.report import (
report_csp_violations,
)
from zerver.views.report import report_csp_violations
from zerver.views.scheduled_messages import (
create_scheduled_message_backend,
delete_scheduled_messages,
Expand Down

0 comments on commit e0ba641

Please sign in to comment.