Skip to content

update docstring in hdulist.py - #7450

Merged
bsipocz merged 2 commits into
astropy:masterfrom
leahmfulmer:update-memmap-docs
May 13, 2018
Merged

bsipocz merged 2 commits into
astropy:masterfrom
leahmfulmer:update-memmap-docs

Conversation

@leahmfulmer

@leahmfulmer leahmfulmer commented May 10, 2018 •

Copy link
Copy Markdown

This supersedes #7433 .

edit: Fix #6497

@astropy-bot

astropy-bot Bot commented May 10, 2018 •

Copy link
Copy Markdown

Hi there @LFulmer 👋 - 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.

@saimn

saimn commented May 11, 2018

Copy link
Copy Markdown
Contributor

@LFulmer - Thanks, but you need to remove the whitespaces at the end of the lines in order to fix the CI build.

astropy/io/fits/hdu/hdulist.py:46:77: W291 trailing whitespace
astropy/io/fits/hdu/hdulist.py:47:48: W291 trailing whitespace

@pllim

pllim commented May 11, 2018

Copy link
Copy Markdown
Member

@saimn @LFulmer , I removed trailing whitespace for you. FYI.

@bsipocz

bsipocz commented May 13, 2018

Copy link
Copy Markdown
Member

Thanks @LFulmer!

@bsipocz
bsipocz merged commit d708ade into astropy:master May 13, 2018
bsipocz added a commit that referenced this pull request May 21, 2018
bsipocz added a commit that referenced this pull request May 21, 2018
bsipocz added a commit that referenced this pull request May 22, 2018
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