Skip to content

Use full path to API link#11

Merged
keflavich merged 2 commits into
keflavich:hitranfrom
migueldvb:hi
Nov 4, 2016
Merged

Use full path to API link#11
keflavich merged 2 commits into
keflavich:hitranfrom
migueldvb:hi

Conversation

@migueldvb

Copy link
Copy Markdown

Thanks to @bsipocz for noticing this problem.

@keflavich keflavich merged commit 3dc7c30 into keflavich:hitran Nov 4, 2016
keflavich added a commit that referenced this pull request May 3, 2026
# This is the 1st commit message:

add HiGal module

more cleanup

add changelog

it is possible for a cutout to be missing; we need to allow for that.
Also, the hack I had before isn't needed when you appropriately query
for multiple wavelengths

remove some comments, still not done with the PR

minor correction

added basic docs

pep8 fixes.  The bracket closures DO match the indentation, though.
pep8 lies sometimes.

add local tests and cleanup some testing

more interactive tests revealed that my hacks didn't really work, this
commit has some fixes

move everything into astroquery.herschel.higal

update relative paths

whitespace

# This is the commit message #2:

update tests

# This is the commit message #3:

the higal server genuinely isn't showing anything when I search online now

# This is the commit message #4:

add HiGal module

# This is the commit message #5:

more cleanup

# This is the commit message #6:

add changelog

# This is the commit message #7:

it is possible for a cutout to be missing; we need to allow for that.
Also, the hack I had before isn't needed when you appropriately query
for multiple wavelengths

# This is the commit message #8:

remove some comments, still not done with the PR

# This is the commit message #9:

minor correction

# This is the commit message #10:

added basic docs

# This is the commit message #11:

pep8 fixes.  The bracket closures DO match the indentation, though.
pep8 lies sometimes.

# This is the commit message #12:

add local tests and cleanup some testing

# This is the commit message #13:

more interactive tests revealed that my hacks didn't really work, this
commit has some fixes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants