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

Blandat 6/1 -25 #737

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Conversation

phieri
Copy link
Collaborator

@phieri phieri commented Dec 30, 2024

Innehåll

Blandade ändringar 2025-01-06. Se respektive commit.

Checklista

  • Följer stilmallen specificerad i CONTRIBUTING.md
  • Bygger utan fel lokalt
  • Bygger utan fel på byggserver
  • Signifikanta förändringar införda i CHANGELOG.md
  • Författaren är klar och godkänner sammanfogning
  • Språkligt granskad (stavning och grammatik)
  • Godkänd av två granskare

@phieri phieri force-pushed the blandat241230 branch 3 times, most recently from d2f4608 to db94e10 Compare January 2, 2025 20:04
@phieri phieri changed the title Blandat241230 Blandat 2/1 -25 Jan 2, 2025
@phieri phieri force-pushed the blandat241230 branch 2 times, most recently from e4317cd to e85e0db Compare January 6, 2025 10:36
@phieri phieri changed the title Blandat 2/1 -25 Blandat 6/1 -25 Jan 6, 2025
@phieri phieri marked this pull request as ready for review January 6, 2025 11:24
@phieri phieri requested review from fnadde42 and sm5phu January 6, 2025 11:25
@phieri phieri requested a review from k7per January 6, 2025 11:25
@@ -92,3 +92,29 @@ installeras.

<kbd>sudo apt install texlive texlive-extra-utils texlive-lang-european
texlive-science texlive-fonts-recommended texlive-fonts-extra latexmk</kbd>

# Använda GitHub Codespaces
Copy link
Collaborator

Choose a reason for hiding this comment

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

Superbra med den här delen! ✨

\newcommand{\hilight}[1]{\colorbox{yellow}{#1}}

% (Beskrivning saknas.)
% Paket för att hantera avsnittsrubriker.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Det är riktigt viktigt att dokumentera i koden så att det blir tydligt för folk framöver att läsa den. Bra att du lagt till det här 👍

Copy link
Collaborator

@fnadde42 fnadde42 left a comment

Choose a reason for hiding this comment

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

Ser bra ut med en fråga 👍

@@ -15,7 +15,7 @@ \chapter{Svensk frekvensplan}
\SIrange{432}{438}{\mega\hertz}.

Med stöd av SFS~2022:511 har Post- och telestyrelsen (PTS)
\emph{Förordning om elektronisk kommunikation} \cite{SFS2022:511}
\emph{Förordning om elektronisk kommunikation}~\cite{SFS2022:511}
publicerat PTSFS~2019:1 \emph{Allmänna råd om den svenska frekvensplanen}
\cite{PTSFS2019:1}.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Missades inte ett ~ tecken här mellan \emph{Allmänna råd om den svenska frekvensplanen} och \cite{PTSFS2019:1}?

Copy link
Collaborator Author

@phieri phieri Jan 8, 2025

Choose a reason for hiding this comment

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

Det är helt riktigt det du skriver. De här ändringarna är baserade på en sökning efter ␣\cite vilket inte matchar cite med andra sorters blanksteg eller radbrytningar före.

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