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

Current development branch #35

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Current development branch #35

wants to merge 6 commits into from

Conversation

mrossinek
Copy link
Member

Consolidates the work on:

docs/NFF-0.1-spec.norg Outdated Show resolved Hide resolved
docs/NFF-0.1-spec.norg Outdated Show resolved Hide resolved
docs/NFF-0.1-spec.norg Outdated Show resolved Hide resolved
docs/NFF-0.1-spec.norg Outdated Show resolved Hide resolved
docs/NFF-0.1-spec.norg Outdated Show resolved Hide resolved
docs/NFF-0.1-spec.norg Outdated Show resolved Hide resolved
docs/NFF-0.1-spec.norg Outdated Show resolved Hide resolved
docs/NFF-0.1-spec.norg Outdated Show resolved Hide resolved
docs/NFF-0.1-spec.norg Outdated Show resolved Hide resolved
docs/NFF-0.1-spec.norg Outdated Show resolved Hide resolved
docs/NFF-0.1-spec.norg Outdated Show resolved Hide resolved
docs/NFF-0.1-spec.norg Outdated Show resolved Hide resolved
docs/NFF-0.1-spec.norg Outdated Show resolved Hide resolved
docs/NFF-0.1-spec.norg Outdated Show resolved Hide resolved
docs/NFF-0.1-spec.norg Outdated Show resolved Hide resolved
grammar.js Outdated Show resolved Hide resolved
This was referenced Apr 3, 2022
mrossinek and others added 6 commits December 9, 2022 15:07
This grammar generation is still quite the mess but I have not found a
better way yet... The parser size is now at 33MB but we have quite some
odd cases here. Especially problematic is the nesting levels being
coupled between the parent object of the slide and its contents.

E.g.:

```norg
-- :
   > this does not work because the outer list item is level 2, so the
     inner quote item also needs to be at least that level
```

Also:

```norg
> :
  --- this also does not work because allowing arbitrary nesting levels
      in here results in *absolutely frickin massive* parsers 😱
```

I think we need to figure out a better way on how to deal with
infectability if we want to implement this properly.
The last clause was overriding defaultPackage with its system-specific attributes
completely.

Instead of `nix build --impure .`, the impure making builtins.currentSystem available,
users should now be able to simply `nix build .` and make the flake importable in pure
contexts.
@vhyrro vhyrro marked this pull request as ready for review January 2, 2023 16:58
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.

4 participants