Conversation
|
please rebase |
|
@notjagan Have you rebased yet? |
|
@physikerwelt I have now merged the branch to reflect the changes made in #46 and updated the code to work with pegJS 0.10.0. If/when this gets merged in, I will then create a pull request to similarly add |
|
@physikerwelt If there is nothing wrong with this pull request, could you merge it please. |
lib/astutil.js
Outdated
| match(target, '\\end{'+t+'}') || | ||
| matrix_has(m); | ||
| }, | ||
| MULTLINE: function(target, t, m) { |
There was a problem hiding this comment.
what is the difference in the functions for multine and matrix. Can't you just create one functaion and assign it to both keys MATRIX and MULTILINE
|
@HowardCohl what is the difference between @notjagan can you take a moment to investigate |
|
is this ready to be merged? |
This is simply an intermediate pull request for before #46 is merged. I do not expect this to pass the checks because it will once this is merged in and the other request is merged in since the other request fixes the checks itself.