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

Organize #198

Merged
merged 20 commits into from Jan 4, 2015
Merged

Organize #198

merged 20 commits into from Jan 4, 2015

Conversation

vermiculus
Copy link
Owner

Cleaning up old branches. These are non-functional changes.

I've got to say though – that was one heck of a merge.

@vermiculus
Copy link
Owner Author

Note that I rebased as part of the merge. It was just too much to go through.

@@ -65,7 +74,7 @@ what you are doing!")
upvote
(unanswered my-tags)))
"List of methods that require auth.
Methods are of form (METHOD SUBMETHODS) where SUBMETHODS
Methods are of form (METHOD . SUBMETHODS) where SUBMETHODS
Copy link
Collaborator

Choose a reason for hiding this comment

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

of THE form?

@Malabarba
Copy link
Collaborator

sx-favorites has some excessive whitespace on line 45 (after the or), can we fix that too?

;; - Unanswered :: Unanswered questions
;; - Unanswered My-tags :: Unanswered questions (subscribed tags)
;; - Featured :: Featured questions
;; - Starred :: Favorite questions
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we're missing a ;;; Code: after this line.

Copy link
Owner Author

Choose a reason for hiding this comment

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

Whoops – you're right.

@Malabarba
Copy link
Collaborator

Ok, I'm good with this.

vermiculus added a commit that referenced this pull request Jan 4, 2015
@vermiculus vermiculus merged commit d3417ee into master Jan 4, 2015
@vermiculus vermiculus deleted the organize branch January 4, 2015 23:11
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

2 participants