Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Acronyms #2

Merged
merged 4 commits into from
Apr 1, 2017
Merged

Acronyms #2

merged 4 commits into from
Apr 1, 2017

Conversation

jbae11
Copy link
Contributor

@jbae11 jbae11 commented Mar 31, 2017

Allows \gls{} function for presentations.
Note: #3

@katyhuff
Copy link
Member

katyhuff commented Apr 1, 2017

This looks good. My only requested change is that the aux file should be removed.

I understand now what you were trying to say yesterday! I thought you were telling me it failed to build because you need to install glossaries for it to run. (which was confusing to me because I didn't use the glossaries package in the template). I see now that you meant you were just trying to add acronyms and had to find the acros tex file, not that you had to actually install anything. Glad it wasn't a platform issue or anything wrong with the template.

Please note that I try to maintain a master list of useful nuclear acronyms here and pull requests are always welcome.

acros.aux Outdated
@@ -0,0 +1,23 @@
\relax
Copy link
Member

Choose a reason for hiding this comment

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

aux files should not be version controlled (they are generated by latex from the .tex files). This should be removed from the PR.

@katyhuff katyhuff merged commit 274454a into arfc:master Apr 1, 2017
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