Skip to content

Conversation

@nabobalis
Copy link
Member

No description provided.

@nabobalis nabobalis added the No Changelog Entry Needed Skip all changelog checks. label May 24, 2022
@pep8speaks

This comment was marked as duplicate.

@nabobalis nabobalis merged commit 809317a into main May 24, 2022
@nabobalis nabobalis deleted the ci branch May 24, 2022 23:52
Copy link
Member

@ConorMacBride ConorMacBride left a comment

Choose a reason for hiding this comment

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

There seems to be lots of broken sentences at the end of doc strings?

Comment on lines +81 to +82
to this Client.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
to this Client.

Comment on lines +3 to +4
repository.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
repository.

Comment on lines +58 to 59
`RECEIVER_FREQUENCIES`.
Copy link
Member

Choose a reason for hiding this comment

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

Here

Comment on lines +126 to +127
of for low and high frequency receivers.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
of for low and high frequency receivers.

Comment on lines +23 to +24
`data archive <https://solar-radio.gsfc.nasa.gov/data/stereo>`__.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
`data archive <https://solar-radio.gsfc.nasa.gov/data/stereo>`__.

Comment on lines +128 to +129
of for low and high frequency receivers.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
of for low and high frequency receivers.

Comment on lines +57 to 58
`RECEIVER_FREQUENCIES`.
Copy link
Member

Choose a reason for hiding this comment

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

Here

Copy link
Member Author

Choose a reason for hiding this comment

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

All fixed now. Thanks.

@nabobalis
Copy link
Member Author

There seems to be lots of broken sentences at the end of doc strings?

I was experimenting with something, it did not work.

@nabobalis
Copy link
Member Author

I will fix and push to main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No Changelog Entry Needed Skip all changelog checks.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants