Skip to content

Use dynamic import() to remove cyclic dependency in Recipe#485

Merged
n1474335 merged 10 commits intogchq:masterfrom
d98762625:dynamic-import
Feb 11, 2019
Merged

Use dynamic import() to remove cyclic dependency in Recipe#485
n1474335 merged 10 commits intogchq:masterfrom
d98762625:dynamic-import

Conversation

@d98762625
Copy link
Copy Markdown
Member

@d98762625 d98762625 commented Feb 1, 2019 •
edited
Loading

Refactor Recipe to use dynamic import to load operations during execution, rather than have a top-level OpModules import. This will hopefully mean we can remove the extra cleaning before generating the index script for the node-api.

  • Make recipe highlighting work
  • Validate with node-lib

@d98762625 d98762625 changed the title WIP: Use dynamic import() to remove cyclic dependency in Recipe Use dynamic import() to remove cyclic dependency in Recipe Feb 11, 2019
@n1474335 n1474335 merged commit 481f2a4 into gchq:master Feb 11, 2019
BRAVO68WEB pushed a commit to BRAVO68WEB/CyberChef that referenced this pull request May 29, 2022
@Lissy93
⚡ Adds option to prevent saving changes to disk (gchq#485)
76e5a1b
BRAVO68WEB pushed a commit to BRAVO68WEB/CyberChef that referenced this pull request May 29, 2022
@Lissy93
⚡ Adds option to prevent saving changes locally (gchq#485)
a65c3c5
BRAVO68WEB pushed a commit to BRAVO68WEB/CyberChef that referenced this pull request May 29, 2022
@Lissy93
🗃️ Adds options to prevent config saving to schema + docs (gchq#485)
e8cee3b
BRAVO68WEB pushed a commit to BRAVO68WEB/CyberChef that referenced this pull request May 29, 2022
@Lissy93
🛂 Adds access control options to interatvie editor menu (gchq#485)
d5f0c19
BRAVO68WEB pushed a commit to BRAVO68WEB/CyberChef that referenced this pull request May 29, 2022
@Lissy93
🛂 Improvedd access controlls in Store (gchq#485)
eb377d2
BRAVO68WEB pushed a commit to BRAVO68WEB/CyberChef that referenced this pull request May 29, 2022
@Lissy93
🛂 Disallow save local/ disk in JSON editr (gchq#485)
0b07abe
BRAVO68WEB pushed a commit to BRAVO68WEB/CyberChef that referenced this pull request May 29, 2022
@Lissy93
🔀 Merge pull request gchq#488 from Lissy93/FIX/general-issues
23f2c1a 
[FIX] General issues and improvements
Fixes gchq#452
Fixes gchq#454
Fixes gchq#455
Fixes gchq#463
Fixes gchq#479
Fixes gchq#482
Fixes gchq#483
Fixes gchq#485
Fixes gchq#486
Fixes gchq#487
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@d98762625 @n1474335