Skip to content

Cleanup#42

Merged
codebytes merged 8 commits intomainfrom
cleanup
May 7, 2026
Merged

Cleanup#42
codebytes merged 8 commits intomainfrom
cleanup

Conversation

@codebytes
Copy link
Copy Markdown
Owner

This pull request updates the styling for fit titles (titles using Marp's feature) in both the dark and light custom Aspire themes. The main goal is to add side margins to h1 elements containing fit SVGs, preventing them from running edge-to-edge and improving readability.

Fit title styling improvements:

  • Added CSS rules to both custom-aspire-light.css and custom-aspire-dark.css so that section h1 elements containing a fit SVG (svg[data-marp-fitting="svg"]) now have side margins (margin-inline: 4%), with increased margins (margin-inline: 6%) for special section types such as cover, lead, purple, gradient, and invert. [1] [2]
  • Updated comments in the CSS files to explain the rationale for these changes, clarifying the behavior of Marp's fit titles and the need for added margins. [1] [2]

codebytes and others added 8 commits May 7, 2026 12:05
- Remove tracked Java .class files in spring-boot-postgres/aspire/
- Remove tracked Angular .angular/cache/ in dotnet-angular-cosmos/frontend/
- Remove tracked .codegen-hash in Aspire .modules/ scaffolding
- Add .gitignore section covering Python venvs, Java build, frontend caches,
  Aspire runtime artifacts, macOS/IDE junk
- Delete untracked temp/build cruft from OTel validation runs

Note: samples/vite-react-api/ and samples/ts-starter/ untouched (active work).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Append summary of untracked build artifacts, disk cleanup, and
.gitignore polyglot patterns. Document patterns for future runs.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Follow-up to a7e37f2: stage the on-disk deletions of the
sample-local nuget package cache that was accidentally tracked.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Follow-up to a7e37f2: convert unanchored patterns to **/ globs so they
match anywhere in samples/. Add vite.config.{d.ts,js} when source is
vite.config.ts. Also delete already-untracked build artifacts.

- **/.modules/.codegen-hash now matches Aspire codegen files in any sample
- **/vite.config.d.ts and **/vite.config.js catch Vite compilation output
- All Python, Java, Go, .NET, and Node virtualenvs now properly matched
- Deleted 6 .codegen-hash files + vite.config.js build artifacts

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ifacts)

Record insights from deep cleanup pass fixing unanchored .gitignore patterns.
Key learnings: glob anchoring critical for monorepo, Vite compile artifacts
need manual detection, regenerating files must never be tracked.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sister commit to 05446da. The .codegen-hash files in flask, spring-boot,
svelte-go, ts-starter, and vite-react-api were tracked from before the
**/ anchor fix; deleting them on disk without 'git add -u' left them as
unstaged deletions. Stage and remove for good — pattern in .gitignore
covers them going forward.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Added OpenTelemetry dependencies to package.json for tracing and instrumentation.
- Created instrumentation.ts to set up tracing, including document load, fetch, and user interaction instrumentation.
- Updated main.tsx to import the new instrumentation module.
- Defined VITE_ environment variables in vite-env.d.ts for OpenTelemetry configuration.
- Modified vite.config.ts to rewrite OTLP endpoint for browser compatibility and expose necessary environment variables to client code.
- Added TypeScript configuration for apphost in tsconfig.apphost.json.
- Updated presentation slides to reflect changes in the Aspire architecture.
@codebytes codebytes merged commit 9b77e20 into main May 7, 2026
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.

1 participant