Thanks for fixing this Nitesh and Sam!
I wrote that R check and forgot that it depended on the Gencode url. I
had to fix this too for recount
https://github.com/leekgroup/recount/commit/056e81c326143e3748b452988ac15a6bed3b035e.
Best,
Leo
On Mon, Nov 5, 2018 at 3:14 PM Turaga, Nitesh
wrote:
>
>
Hi Sam,
I fixed the RELEASE_3_8 branch as well. There was a minor discrepancy in the
commit history for RELEASE_3_8 though, (which I fixed in
git.bioconductor.org/packages/bumphunter) so you should resync RELEASE_3_8 in
rafalab/bumphunter.
The commit history on rafalab/bumphunter:RELEASE_3_8
Hi Nitesh,
I have fixed the commit history for rafalab/bumphunter RELEASE_3_8
branch, and request synchronization with Bioconductor RELEASE_3_8. We
need this, because bumphunter is not building on RELEASE_3_8, due to an
obsolete URL in one of the tests.
thanks,
- Sam
On 11/5/18 7:56 AM, T
Hi Sam,
I have updated your “master” branch on git.bioconductor.com to reflect what you
have on GitHub, at https://github.com/rafalab/bumphunter. You will not get the
duplicate commit error when you push to the master branch anymore.
However, RELEASE_3_8 still has duplicate commits in the his
Dear Bioconductor,
I believe I have removed all duplicate commits from the bumphunter
package. The main branch in rafalab/bumphunter incorporates the new
commit history. It passes 'library(devtools); build("."); install();
check("."); BiocCheck(".")'
I am contacting you in accordance with it
