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

Update ruler length to 280 characters #391

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

atoponce
Copy link

Per the recent Twitter update from 140-character to 280-character tweet lengths, a 140-character ruler is now obsolete. This supports the new 280-character max tweet length.

@@ -688,10 +688,10 @@ def retweets_of_me
end
map %w(retweetsofme) => :retweets_of_me

desc 'ruler', 'Prints a 140-character ruler'
desc 'ruler', 'Prints a 280-character ruler'
Copy link
Contributor

Choose a reason for hiding this comment

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

You'll also need to update the reference to this in etc/t-completion.zsh — see the diff in #389. This and #389 are dupes of each other, but each one is missing a piece that the other has, so whichever of you is the first to update is probably the PR that should go ¯\_(ツ)_/¯

@tjschuck
Copy link
Contributor

@atoponce Can you squash this down to one commit? Instructions here.

update etc/t-completion.zsh for 140->280 character change
Update ruler length to 280 characters per twitter change
@atoponce
Copy link
Author

I have no idea what I'm doing. Heh. ¯_(ツ)_/¯

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