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

feat(astro/component): adds an inline tag attribute for inline code #5973

Closed
wants to merge 6 commits into from

Conversation

Lofty-Brambles
Copy link

Changes

This adds an inline attribute to render code in the <Code /> component as inline code without the pre tag.
This PR is related to the issue: #5912

Testing

No tests were present for the <Code /> component.

Docs

Is this, along with the other attributes necessary to be documented? The current docs on it is pretty barebones. I'm willing to add a PR, if necessary.
/cc @withastro/maintainers-docs for feedback!

@changeset-bot
Copy link

changeset-bot bot commented Jan 25, 2023

🦋 Changeset detected

Latest commit: 56fa652

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Lofty-Brambles Lofty-Brambles changed the title [astro/component] feat(astro/component): adds an inline tag attribute for inline code Jan 25, 2023
@github-actions github-actions bot added the pkg: astro Related to the core `astro` package (scope) label Jan 25, 2023
@matthewp
Copy link
Contributor

Looks like your branch has some conflicts. I would fix those and rebase with main which I think will take care of your build failing as well.

@github-actions github-actions bot added the feat: markdown Related to Markdown (scope) label Jan 26, 2023
@Lofty-Brambles
Copy link
Author

That should do it? The ubuntu-latest build is failing due to a type mismatch between string and an union of predefined shiki themes.

@Lofty-Brambles
Copy link
Author

This issue seems to arise from


Which is resolving to string | IShikiTheme instead of Theme | IShikiTheme. Not sure as to how to fix this though- particularly why BUNDLED_THEMES is typed like [Theme, ...Theme[]]

@matthewp
Copy link
Contributor

Why would that be causing build issues? Your change doesn't touch that file?

Oh, is it just because you updated the Shiki version?

@Lofty-Brambles
Copy link
Author

Yep, the bump seems to be the reason

@matthewp
Copy link
Contributor

matthewp commented Mar 7, 2023

Closing as this is stale. The changes look good! Please resubmit when you get the time.

@matthewp matthewp closed this Mar 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat: markdown Related to Markdown (scope) pkg: astro Related to the core `astro` package (scope)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants