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

bugfix for adding the newest package again, while selecting "archive"… #97

Merged
merged 2 commits into from Jul 1, 2020

Conversation

@FelixErnst
Copy link
Contributor

@FelixErnst FelixErnst commented Jul 1, 2020

… as action

zero number of packages are removed in this case and updateRepo throws an error as expected.

This escaped my tests and I stumbled upon it by accident. test for this is added as well.

FelixErnst added 2 commits Jul 1, 2020
… as action

zero number of packages are removed in this case and updateRepo throws an error as expected.
@eddelbuettel
Copy link
Owner

@eddelbuettel eddelbuettel commented Jul 1, 2020

Nice catch.

I still appreciate ChangeLog entries too :) (but I don't mind adding them ex-post so need for a PR just for that)

@eddelbuettel eddelbuettel merged commit b4d5c31 into eddelbuettel:master Jul 1, 2020
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@FelixErnst
Copy link
Contributor Author

@FelixErnst FelixErnst commented Jul 1, 2020

will do!

@FelixErnst FelixErnst deleted the FelixErnst:bugfixes branch Jul 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.