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

Change the way "Is the classes correct?" is checked #2

Closed
wants to merge 2 commits into from
Closed

Change the way "Is the classes correct?" is checked #2

wants to merge 2 commits into from

Conversation

ickc
Copy link

@ickc ickc commented May 8, 2016

Related to what I described in ickc/pandoc-amsthm#2

Hope that I didn't misunderstand what you want to do.

@coveralls
Copy link

coveralls commented May 8, 2016

Coverage Status

Coverage increased (+0.2%) to 93.103% when pulling cd36915 on ickc:master into d521150 on chdemko:master.

@chdemko
Copy link
Owner

chdemko commented May 9, 2016

In my code, the lists have been converted to set using the built-in python set function. I expect that the current set is a super set of the defined set. Your code check only if a class is present to add the LaTeX code.

@ickc
Copy link
Author

ickc commented May 9, 2016

I see. I understood what it is doing, but just don't know it is intentional. Perhaps some more elaborated examples on the use case?

Thanks and I'm closing it.

@ickc ickc closed this May 9, 2016
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.

None yet

3 participants