Skip to content

Add orchardcore-asset-manager skill#19154

Open
Skrypt wants to merge 2 commits intomainfrom
skrypt/asset-manager-skill
Open

Add orchardcore-asset-manager skill#19154
Skrypt wants to merge 2 commits intomainfrom
skrypt/asset-manager-skill

Conversation

@Skrypt
Copy link
Copy Markdown
Contributor

@Skrypt Skrypt commented Apr 15, 2026

Documents build commands, Node.js version handling, all Assets.json action types, the three-tier package structure, and code quality commands (lint, check, dry-run).

Skrypt and others added 2 commits April 15, 2026 15:40
Documents build commands, Node.js version handling, all Assets.json
action types, the three-tier package structure, and code quality
commands (lint, check, dry-run).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Explains that .claude/settings.json is intentionally gitignored and
should be created locally by each developer, with the reason why it
must not be committed.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@Skrypt Skrypt requested a review from Piedone April 15, 2026 19:51
@Piedone
Copy link
Copy Markdown
Member

Piedone commented Apr 15, 2026

I won't be able to review this in the foreseeable future, I'm sorry.

@Skrypt
Copy link
Copy Markdown
Contributor Author

Skrypt commented Apr 15, 2026

ah yeah no javascript right

@Skrypt
Copy link
Copy Markdown
Contributor Author

Skrypt commented Apr 15, 2026

No one can review this other than me and @jptissot

@Skrypt Skrypt requested review from MikeAlhayek and jptissot and removed request for Piedone and jptissot April 15, 2026 20:01
@Skrypt
Copy link
Copy Markdown
Contributor Author

Skrypt commented Apr 17, 2026

I'm guessing this is the kind of PR no one will really be able to approve other than me. 😄

@jptissot
Copy link
Copy Markdown
Member

:) Nice. I will try to find the time to review this. Been pretty busy lately though.

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.

3 participants