Skip to content

feat: sharkymark profile and claude docker template #304

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

Merged
merged 9 commits into from
Aug 9, 2025

Conversation

sharkymark
Copy link
Contributor

@sharkymark sharkymark commented Aug 7, 2025

Closes #

Description

registry profile creation and template submission for claude on docker

Type of Change

  • New template
  • Bug fix
  • Feature/enhancement
  • Documentation
  • Other

Template Information

Path: registry/sharkymark/templates/docker-claude
New version: v1.0.0
Breaking change: [ ] Yes [x] No

Testing & Validation

  • Tests pass (bun test)
  • Code formatted (bun run fmt)
  • Changes tested locally

Related Issues

n/a

@DevelopmentCats
Copy link
Contributor

This module essentially mimics the existing tasks-docker template available on https://registry.coder.com/templates/coder-labs/tasks-docker

Copy link
Member

@matifali matifali left a comment

Choose a reason for hiding this comment

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

Thanks a lot for the contribution @sharkymark.

@matifali matifali reopened this Aug 8, 2025
Copy link
Member

@bpmct bpmct left a comment

Choose a reason for hiding this comment

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

Mark, this is awesome! Welcome to the Registry :)

@DevelopmentCats
Copy link
Contributor

DevelopmentCats commented Aug 8, 2025

Sorry for closing this earlier. Thank you for the contribution!

the last thing you might need to do is rename the readme in your namespace from readme.MD to README.md so the CI for README validation will run @sharkymark

@sharkymark
Copy link
Contributor Author

@matifali
@bpmct @sharkymark thanks for the catch. I made the changes.

It was early morning yesterday and hadn't had my first Celcius yet. 😂

Wanted to call out @phorcys420 for encouraging me this week to do this, and @matifali for the original encouragement.

Good luck Coder 🦈

@sharkymark sharkymark requested review from bpmct and matifali August 8, 2025 13:42
@DevelopmentCats
Copy link
Contributor

DevelopmentCats commented Aug 9, 2025

I downloaded this template to try and build it in my coder deployment. @sharkymark

Its failing with this response:

{
    "message": "Unable to validate presets",
    "validations": [
        {
            "field": "Real World App: Angular + Django",
            "detail": "Undefined Parameter; Preset parameter \"jetbrains_ide\" is not defined by the template."
        }
    ]
}

You can probably just remove this if you don't plan on including the jetbrains module in the template.

@sharkymark
Copy link
Contributor Author

Something must have changed in the next release. I'll give it a whirl and report back ☮️

@sharkymark
Copy link
Contributor Author

@DevelopmentCats I removed reference to JetBrains and tested.

@bpmct bpmct enabled auto-merge (squash) August 9, 2025 16:32
@bpmct bpmct requested a review from phorcys420 August 9, 2025 16:39
@bpmct bpmct merged commit 59b0472 into coder:main Aug 9, 2025
4 checks passed
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.

5 participants