strip trailing whitespace and add missing final newlines#36
Open
averms wants to merge 1 commit into
Open
Conversation
Removes trailing whitespace (all single trailing spaces in prose, no Markdown hard breaks affected) and adds missing final newlines across 21 files.
Contributor
There was a problem hiding this comment.
Pull request overview
This PR performs a repository-wide Markdown hygiene pass, removing trailing whitespace and ensuring missing final newlines are present across a set of skill guides and reference docs.
Changes:
- Strip trailing whitespace from prose and headings (without altering Markdown hard breaks).
- Add missing final newlines to affected Markdown and text files.
- Keep shared anywidget reference copies aligned while applying whitespace-only edits.
Reviewed changes
Copilot reviewed 1 out of 21 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| skills/streamlit-to-marimo/SKILL.md | Trailing-whitespace cleanup in guidance text and headings |
| skills/marimo-notebook/SKILL.md | Trailing-whitespace cleanup; adds final newline where needed |
| skills/marimo-notebook/references/WATCHING.md | Trailing-whitespace cleanup; adds final newline where needed |
| skills/marimo-notebook/references/UI.md | Trailing-whitespace cleanup; adds final newline where needed |
| skills/marimo-notebook/references/TOP-LEVEL-IMPORTS.md | Ensures final newline / code fence formatting consistency |
| skills/marimo-notebook/references/SQL.md | Trailing-whitespace cleanup; adds final newline where needed |
| skills/marimo-notebook/references/PYTEST.md | Trailing-whitespace cleanup; adds final newline where needed |
| skills/marimo-notebook/references/EXPORTS.md | Trailing-whitespace cleanup; adds final newline where needed |
| skills/marimo-notebook/references/DEPLOYMENT.md | Trailing-whitespace cleanup; adds final newline where needed |
| skills/marimo-notebook/references/COLUMNS.md | Trailing-whitespace cleanup; adds final newline where needed |
| skills/marimo-notebook/references/ANYWIDGET.md | Trailing-whitespace cleanup in synced anywidget reference |
| skills/marimo-batch/SKILL.md | Trailing-whitespace cleanup across examples and prose |
| skills/jupyter-to-marimo/references/widgets.md | Trailing-whitespace cleanup; adds final newline where needed |
| skills/implement-paper/SKILL.md | Trailing-whitespace cleanup; adds final newline where needed |
| skills/implement-paper/references/anywidget.md | Trailing-whitespace cleanup in synced anywidget reference |
| skills/implement-paper-auto/SKILL.md | Trailing-whitespace cleanup; adds final newline where needed |
| skills/implement-paper-auto/references/ANYWIDGET.md | Trailing-whitespace cleanup; adds final newline where needed |
| skills/auto-paper-demo/SKILL.md | Trailing-whitespace cleanup; adds final newline where needed |
| skills/auto-paper-demo/references/ANYWIDGET.md | Trailing-whitespace cleanup; adds final newline where needed |
| skills/anywidget/SKILL.md | Trailing-whitespace cleanup in anywidget source skill doc |
| .gitignore | Trailing-whitespace cleanup in comment lines |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ## Responding to other files | ||
|
|
||
| marimo has `mo.watch.file` and `mo.watch.file` utilities that can cause cells to update when a file/folder updates. | ||
| marimo has `mo.watch.file` and `mo.watch.file` utilities that can cause cells to update when a file/folder updates. |
| ``` | ||
|
|
||
| `grouped` is a polars dataframe. By defauly marimo uses DuckDB in memory and can refer to dataframe variables that are in scope. | ||
| `grouped` is a polars dataframe. By defauly marimo uses DuckDB in memory and can refer to dataframe variables that are in scope. |
| ## Step 2: Fetch the paper | ||
|
|
||
| If the user gives you an Arxiv/AlphaXiv link, you will an efficient way to read the paper. | ||
| If the user gives you an Arxiv/AlphaXiv link, you will an efficient way to read the paper. |
| This is the best of both worlds, you can use the UI to test and iterate, and then use the CLI to run the batch job. Another benefit is that you can run the notebook with settings to make it run quickly to see if there are any bugs in the notebook. | ||
|
|
||
| The user wants to be able to run a notebook using this pattern, so make sure you ask the user which parameters they want to make configurable via the CLI and the proceed to make the changes to the notebook. Make sure you verify the changes with the user before making them. | ||
| The user wants to be able to run a notebook using this pattern, so make sure you ask the user which parameters they want to make configurable via the CLI and the proceed to make the changes to the notebook. Make sure you verify the changes with the user before making them. |
| ## Weights and Biases | ||
|
|
||
| It is possible that the user is interested in adding support for weights and biases. Make sure you confirm if this is the case yes/no. If that is the case, make sure these ModelParams are logged. You also want to make sure that the `wandb_project` and `wandb_run_name` are part of the ModelParams is the user wants to go down this route. | ||
| It is possible that the user is interested in adding support for weights and biases. Make sure you confirm if this is the case yes/no. If that is the case, make sure these ModelParams are logged. You also want to make sure that the `wandb_project` and `wandb_run_name` are part of the ModelParams is the user wants to go down this route. |
| Commands: | ||
| html Run a notebook and export it as an HTML file. | ||
| html-wasm Export a notebook as a WASM- powered marimo notebook. | ||
| html-wasm Export a notebook as a WASM- powered marimo notebook. |
| ``` | ||
|
|
||
| Notice how the notebook is structured with functions can represent cell contents. Each cell is defined with the `@app.cell` decorator and the inputs/outputs of the function are the inputs/outputs of the cell. marimo usually takes care of the dependencies between cells automatically. | ||
| Notice how the notebook is structured with functions can represent cell contents. Each cell is defined with the `@app.cell` decorator and the inputs/outputs of the function are the inputs/outputs of the cell. marimo usually takes care of the dependencies between cells automatically. |
| For more information on testing with pytest see [PYTEST.md](references/PYTEST.md) | ||
|
|
||
| Once tests are added, you can run pytest from the commandline on the notebook to run pytest. | ||
| Once tests are added, you can run pytest from the commandline on the notebook to run pytest. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Removes trailing whitespace (all single trailing spaces in prose, no Markdown hard breaks affected) and adds missing final newlines across 21 files.