-
Notifications
You must be signed in to change notification settings - Fork 12
Handle echo
option to fix #309
#311
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
base: main
Are you sure you want to change the base?
Conversation
In a bit more detail: ---
engine: julia
---
```{julia}
#| echo: false
using PythonCall
```
```{python}
print("option `echo:`")
```
```{python}
#| echo: false
print("option `echo: false`")
```
```{python}
#| echo: true
print("option `echo: true`")
```
```{python}
#| echo: fenced
print("option `echo: fenced`")
``` {
"metadata": {
"kernelspec": {
"display_name": "Julia 1.11.5",
"language": "julia",
"name": "julia-1.11.5"
},
"kernel_info": {
"name": "julia"
},
"language_info": {
"name": "julia",
"version": "1.11.5",
"codemirror_mode": "julia"
}
},
"nbformat": 4,
"nbformat_minor": 5,
"cells": [
{
"id": "1",
"cell_type": "markdown",
"metadata": {
},
"source": [
"---\n",
"engine: julia\n",
"---"
]
},
{
"id": "2",
"cell_type": "code",
"metadata": {
},
"source": [
"#| echo: false\n",
"using PythonCall"
],
"outputs": [
],
"execution_count": 1
},
{
"id": "3",
"cell_type": "markdown",
"metadata": {
},
"source": [
]
},
{
"id": "4_code_prefix",
"cell_type": "markdown",
"metadata": {
},
"source": [
"```python\n",
"print(\"option `echo:`\")\n",
"```"
]
},
{
"id": "4",
"cell_type": "code",
"metadata": {
},
"source": [
"#| echo: false\n",
"print(\"option `echo:`\")"
],
"outputs": [
{
"output_type": "stream",
"name": "stdout",
"text": "option `echo:`\n"
}
],
"execution_count": 1
},
{
"id": "5",
"cell_type": "markdown",
"metadata": {
},
"source": [
]
},
{
"id": "6_code_prefix",
"cell_type": "markdown",
"metadata": {
},
"source": [
]
},
{
"id": "6",
"cell_type": "code",
"metadata": {
},
"source": [
"#| echo: false\n",
"print(\"option `echo: false`\")"
],
"outputs": [
{
"output_type": "stream",
"name": "stdout",
"text": "option `echo: false`\n"
}
],
"execution_count": 1
},
{
"id": "7",
"cell_type": "markdown",
"metadata": {
},
"source": [
]
},
{
"id": "8_code_prefix",
"cell_type": "markdown",
"metadata": {
},
"source": [
"```python\n",
"print(\"option `echo: true`\")\n",
"```"
]
},
{
"id": "8",
"cell_type": "code",
"metadata": {
},
"source": [
"#| echo: false\n",
"print(\"option `echo: true`\")"
],
"outputs": [
{
"output_type": "stream",
"name": "stdout",
"text": "option `echo: true`\n"
}
],
"execution_count": 1
},
{
"id": "9",
"cell_type": "markdown",
"metadata": {
},
"source": [
]
},
{
"id": "10_code_prefix",
"cell_type": "markdown",
"metadata": {
},
"source": [
"```{{python}}\n",
"print(\"option `echo: fenced`\")\n",
"```"
]
},
{
"id": "10",
"cell_type": "code",
"metadata": {
},
"source": [
"#| echo: false\n",
"print(\"option `echo: fenced`\")"
],
"outputs": [
{
"output_type": "stream",
"name": "stdout",
"text": "option `echo: fenced`\n"
}
],
"execution_count": 1
},
{
"id": "11",
"cell_type": "markdown",
"metadata": {
},
"source": [
]
}
]
} |
Thanks @denius, when I get some available time I will take a look and the implementation and review. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #311 +/- ##
==========================================
- Coverage 79.40% 79.17% -0.24%
==========================================
Files 39 39
Lines 2064 2089 +25
==========================================
+ Hits 1639 1654 +15
- Misses 425 435 +10 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Without tests, I don't know how to do them.