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

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

Merged
merged 10 commits into from
Feb 11, 2019

Conversation

d98762625
Copy link
Member

@d98762625 d98762625 commented Feb 1, 2019

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
BRAVO68WEB pushed a commit to BRAVO68WEB/CyberChef that referenced this pull request May 29, 2022
BRAVO68WEB pushed a commit to BRAVO68WEB/CyberChef that referenced this pull request May 29, 2022
BRAVO68WEB pushed a commit to BRAVO68WEB/CyberChef that referenced this pull request May 29, 2022
BRAVO68WEB pushed a commit to BRAVO68WEB/CyberChef that referenced this pull request May 29, 2022
BRAVO68WEB pushed a commit to BRAVO68WEB/CyberChef that referenced this pull request May 29, 2022
BRAVO68WEB pushed a commit to BRAVO68WEB/CyberChef that referenced this pull request May 29, 2022
[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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants