Changed cat() to message() so that I may suppressMessage() these call… #12
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…s when trying many iterations. Before I realized, my editor removed some trailing whitespaces and changed the indentation of one file, I stopped it from being more invasive, but emacs wants desperately to standardize the indentation of all the files and remove the tab characters hiding in them.
|
FYI, you could use |
|
I'm happy to make this change, but would need a pull request where there are no conflicts to merge. |
|
No worries. I am redoing the pull and commits now and will send along
separate pushes with the white space changes (along with a diff) and
then the actual changes and a separate diff.
hmm I wonder if this will go to you, I do not think I have ever
corresponded with anyone via the github emails -- if not then I hope you
are not annoyed by a couple of random pull requests in a few minutes.
…On 12/13/2016 01:01 PM, Jeff L. wrote:
I'm happy to make this change, but would need a pull request where there
are no conflicts to merge.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#12 (comment)>, or
mute the thread
<https://github.com/notifications/unsubscribe-auth/AAEA7QJtwjhyTZpNqtxm1EEuaeYz7d9yks5rHt1hgaJpZM4IWYTS>.
--
=================
Ashton Trey Belew
Gain some confidence
That you found me by my key.
Compare it against:
https://pgp.mit.edu/pks/lookup?op=vindex&search=0x17D50871E48861EA
|
|
Hi,
Attached is a diff of just whitespace shenanigans.
…On 12/13/2016 01:01 PM, Jeff L. wrote:
I'm happy to make this change, but would need a pull request where there
are no conflicts to merge.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#12 (comment)>, or
mute the thread
<https://github.com/notifications/unsubscribe-auth/AAEA7QJtwjhyTZpNqtxm1EEuaeYz7d9yks5rHt1hgaJpZM4IWYTS>.
--
=================
Ashton Trey Belew
Gain some confidence
That you found me by my key.
Compare it against:
https://pgp.mit.edu/pks/lookup?op=vindex&search=0x17D50871E48861EA
|
|
Attached is a diff which converts cat() to message() in a way which is
hopefully not annoying. Please note that I did not load this into a
running R session to make certain that I did not make any typographical
mistakes.
I hope you have a great day, sva is neat!
atb
…On 12/13/2016 01:01 PM, Jeff L. wrote:
I'm happy to make this change, but would need a pull request where there
are no conflicts to merge.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#12 (comment)>, or
mute the thread
<https://github.com/notifications/unsubscribe-auth/AAEA7QJtwjhyTZpNqtxm1EEuaeYz7d9yks5rHt1hgaJpZM4IWYTS>.
--
=================
Ashton Trey Belew
Gain some confidence
That you found me by my key.
Compare it against:
https://pgp.mit.edu/pks/lookup?op=vindex&search=0x17D50871E48861EA
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
…s when trying many iterations.
I hope this is not too presumptuous, but I wanted to be able to
suppressMessage() my invocations of sva.
Before I realized, my editor removed some trailing whitespaces and changed the indentation of one file, I stopped it from being more invasive, but emacs wants desperately to standardize the indentation of all the files and remove the tab characters hiding in them.