Skip to content
This repository has been archived by the owner on Mar 23, 2023. It is now read-only.

Changes to address review from editors #8

Merged
merged 9 commits into from
Jan 28, 2023
Merged

Changes to address review from editors #8

merged 9 commits into from
Jan 28, 2023

Conversation

nabobalis
Copy link
Contributor

No description provided.

sections/2-code.tex Outdated Show resolved Hide resolved
sections/2-code.tex Outdated Show resolved Hide resolved
sections/2-code.tex Outdated Show resolved Hide resolved
sections/2-code.tex Outdated Show resolved Hide resolved
@nabobalis nabobalis marked this pull request as ready for review January 20, 2023 23:36
sections/2-code.tex Outdated Show resolved Hide resolved
sections/2-code.tex Outdated Show resolved Hide resolved
@@ -20,6 +20,8 @@
\newcommand{\astropy}{Astropy\xspace}
\newcommand{\astropypkg}{\package{astropy}}
\newcommand{\aiapypkg}{\package{aiapy}}
\newcommand{\pfsspypkg}{\package{pfsspy}}
\newcommand{\soarpkg}{\package{sunpy\_soar}}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah I should have escaped it. Doh

@nabobalis
Copy link
Contributor Author

precommit unhappy:

sections/2-code.tex:9:While other packages in the ecosystem may focus on particular analysis techniques or analyzing data from specific instruments, the \sunpypkg ``core'' package is focused on providing general tools for working with solar physics data.

Copy link
Contributor Author

@nabobalis nabobalis left a comment

Choose a reason for hiding this comment

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

Thanks Will 👍

sections/2-code.tex Outdated Show resolved Hide resolved
sections/2-code.tex Outdated Show resolved Hide resolved
Co-authored-by: David Stansby <dstansby@gmail.com>
@nabobalis
Copy link
Contributor Author

We can just ignore the pre commit, enquote and added don't seem to play well together.

@wtbarnes
Copy link
Member

Lol yeah sorry I should've stuck that in a comment. I had to move those to the usual LaTeX quote style because of the track changes markup.

@wtbarnes wtbarnes merged commit 8d9f83a into main Jan 28, 2023
@nabobalis nabobalis deleted the review branch January 29, 2023 00:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants