Skip to content

Remove warning when overwriting files with BinTableHDU.dump - #8045

Merged
pllim merged 1 commit into
astropy:masterfrom
saimn:fits-dump-warning
Oct 31, 2018
Merged

pllim merged 1 commit into
astropy:masterfrom
saimn:fits-dump-warning

Conversation

@saimn

@saimn saimn commented Oct 30, 2018

Copy link
Copy Markdown
Contributor

Following #7998 (comment), I think this warning is not needed. Write functions usually do not warn when overwriting files.

@astropy-bot

astropy-bot Bot commented Oct 30, 2018

Copy link
Copy Markdown

Hi there @saimn 👋 - thanks for the pull request! I'm just a friendly 🤖 that checks for issues related to the changelog and making sure that this pull request is milestoned and labeled correctly. This is mainly intended for the maintainers, so if you are not a maintainer you can ignore this, and a maintainer will let you know if any action is required on your part 😃.

Everything looks good from my point of view! 👍

If there are any issues with this message, please report them here.

@drdavella drdavella left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@drdavella

Copy link
Copy Markdown
Contributor

Test failure on circleci seems to be unrelated (and is currently affecting all PRs).

@MSeifert04 MSeifert04 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@pllim
pllim merged commit d2f2274 into astropy:master Oct 31, 2018
@saimn
saimn deleted the fits-dump-warning branch October 31, 2018 21:04
bsipocz pushed a commit that referenced this pull request Oct 31, 2018
Remove warning when overwriting files with BinTableHDU.dump
bsipocz pushed a commit that referenced this pull request Nov 8, 2018
Remove warning when overwriting files with BinTableHDU.dump
bsipocz pushed a commit that referenced this pull request Nov 9, 2018
Remove warning when overwriting files with BinTableHDU.dump
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants