Skip to content
This repository was archived by the owner on Nov 12, 2025. It is now read-only.

Play nicely#2

Open
NicholasKobald wants to merge 18 commits into7Geese:masterfrom
NicholasKobald:play-nicely
Open

Play nicely#2
NicholasKobald wants to merge 18 commits into7Geese:masterfrom
NicholasKobald:play-nicely

Conversation

@NicholasKobald
Copy link
Copy Markdown

No description provided.

if not isinstance(list_or_string, (list, tuple, set)):
list_or_string = [list_or_string]
else:
list_or_string = list_or_string
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😆


# audience_activities_query = sql.select(['*']).where(self.activities_table.c.id.in_(audience_query))
# result_proxy = self.engine.execute(audience_activities_query)
"""
Copy link
Copy Markdown

@teeberg teeberg May 23, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

random gotcha: note how triple quotes don't really comment it out, but just make it a string, i.e. a statement that doesn't do anything. except that python concatenates consecutive strings, i.e. this becomes part of the above assertion failure message

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do know that ;) I was just being lazy and wanted the code not to execute

Copy link
Copy Markdown

@teeberg teeberg May 24, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you know about Cmd+/? 😊

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can select a block of code and it'll comment it out/in for you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants