Skip to content

Conversation

KristofferC
Copy link
Member

Happy for help, you can add new highlights or fill out the existing ones. I've started a bit.

Copy link

Once the build has completed, you can preview your PR at this URL: https://julialang.netlify.app/previews/PR2351/ in ~15 minutes


**Example (before → after)**

```llvm
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that this line is causing CI to fail.

Example CI log.

Could not find the language 'llvm', did you forget to load/include a language module?
/home/runner/work/www.julialang.org/www.julialang.org/_libs/highlight/highlight.min.js:234
;if(!N)throw H(o.replace("{}",e)),Error('Unknown language: "'+e+'"')
       ^

Error: Unknown language: "llvm"
    at _ (/home/runner/work/www.julialang.org/www.julialang.org/_libs/highlight/highlight.min.js:234:35)
    at Object.f [as highlight] (/home/runner/work/www.julialang.org/www.julialang.org/_libs/highlight/highlight.min.js:184:28)
    at [eval]:1:2988
    at Script.runInThisContext (node:vm:129:12)
    at Object.runInThisContext (node:vm:307:38)
    at node:internal/process/execution:79:19
    at [eval]-wrapper:6:22
    at evalScript (node:internal/process/execution:78:60)
    at node:internal/main/eval_string:28:3

@ViralBShah ViralBShah enabled auto-merge (squash) September 16, 2025 04:00
@ViralBShah ViralBShah disabled auto-merge September 16, 2025 04:00
@DilumAluthge
Copy link
Member

Converting to draft while WIP.

@DilumAluthge DilumAluthge marked this pull request as draft September 16, 2025 12:51
@DilumAluthge
Copy link
Member

The check_orgs CI failure can be ignored.

@DilumAluthge
Copy link
Member

The "Build and deploy" CI failure is real - the llvm code highlighting is failing. I've asked for help in the #franklin channel on Slack.


**Example (before → after)**

```llvm
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For now, how about we remove the llvm and just format it as plain text. And then if we can figure out how to get llvm syntax highlighting to work with Franklin.jl, then we can add it back?

Suggested change
```llvm
```

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.

9 participants