Skip to content

IMenu: Add 'Tests' as menu entries #39

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

Closed
wants to merge 5 commits into from
Closed

IMenu: Add 'Tests' as menu entries #39

wants to merge 5 commits into from

Conversation

tomterl
Copy link
Contributor

@tomterl tomterl commented Mar 10, 2014

The important commit is b027891, a very simple and small change to have ExUnit test cases appear in the IMenu.

@tonini
Copy link
Contributor

tonini commented Mar 10, 2014

@tomterl Could you please first clean the commit history? I think If you just make a rebase against the master, the commit 800e2f2 isn't necessary.

syohex and others added 3 commits March 10, 2014 09:01
Changing 'default-tab-width' makes no sense. Only changing 'tab-width'
is enough to changing tab size of buffer.

And 'default-tab-width' causes a byte-compile warning
because it is obsolete variable.
@tomterl
Copy link
Contributor Author

tomterl commented Mar 10, 2014

hmm - I did a 'git rebase upstream/master', but is the result cleaner? I can delete my repo, clone again and apply this small patch and create another PR if not.

Or could you just cherry-pick f5bcd8e ?

@tonini
Copy link
Contributor

tonini commented Mar 10, 2014

If you already made the changes in a separat branch:

First I would update your local master branch

  git fetch upstream
  git merge upstream
  git checkout <branch-with-changes>
  git rebase master

@tomterl
Copy link
Contributor Author

tomterl commented Mar 10, 2014

This was FUBAR, #40 is clean.

@tomterl tomterl closed this Mar 10, 2014
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.

3 participants