search

mhelvens / latex-pkgloader

a LaTeX package for managing the options and loading order of other packages
LaTeX Project Public License v1.3c
33 stars 3 forks source link

Add rule for selnolig #13

Closed koppor closed 6 years ago

koppor commented 6 years ago

In case "babel" is loaded, it has to go before "selnolig". Otherwise, one gets "Option clash". See http://www.tex.ac.uk/FAQ-optionclash.html and https://github.com/mhelvens/latex-pkgloader/pull/18#discussion_r175339714.

"fontspec" has always be loaded before "selnolig". The two expressed rules look the same way, but I think, it expresses only the second rule, doesn't it? What does one need to put? Is this implemented by https://github.com/mhelvens/latex-pkgloader/pull/13/commits/c37326e2ba196fae0bfeb3bc13f695578d4a3f56?

mhelvens commented 6 years ago

I'll see if I can merge all these PRs into a single PR today. Sorry if it's taken a while. I only have a little bit of time during weekends.

koppor commented 6 years ago

One can merge one, resolve conflicts for the next, merge, resolve conflicts, ... 😂😂

(Currently on the road)

Am 24.03.2018 7:07 nachm. schrieb "Michiel Helvensteijn" < notifications@github.com>:

Closed #13 https://github.com/mhelvens/latex-pkgloader/pull/13.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/mhelvens/latex-pkgloader/pull/13#event-1539300710, or mute the thread https://github.com/notifications/unsubscribe-auth/ABTafmwRWm9Uc8SPEpSgWzB1UyDBcgW2ks5thotjgaJpZM4SkI_j .

mhelvens commented 6 years ago

And that's what I did, but I didn't want to merge them into master directly, and this way I only had to test / QA once.