Skip to content

Conversation

@Dominik-Matysiak-gravity
Copy link
Collaborator

No description provided.

Copy link

@amazon-q-developer amazon-q-developer bot left a comment

Choose a reason for hiding this comment

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

Review Summary

This PR updates the workshop numbering from the previous sequence (04, 05, 07) to a more logical sequential order (01, 02, 03, 04, 05). The changes also include some content improvements:

Key Changes:

  • Workshop numbering: Updated titles to follow sequential order (01-05)
  • Marketplace command correction: Fixed the marketplace add command from anthropics/claude-code-plugins to anthropics/claude-code
  • Plugin installation method: Changed from /plugin install ./my-plugin to claude --plugin-dir ./my-plugin
  • Added important note: Added restart requirement for hooks configuration
  • Content cleanup: Removed the "Example: Official Plugin Dev Tools" section

Concerns:

The main concern is the change in plugin installation methodology in 05_plugins.md. Please verify that the new claude --plugin-dir approach is the correct and current way to test local plugins.

Overall, the workshop reorganization and content updates improve the learning flow and accuracy of the documentation.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

Comment on lines -148 to +149
/plugin install ./my-plugin
claude --plugin-dir ./my-plugin

Choose a reason for hiding this comment

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

The change from /plugin install ./my-plugin to claude --plugin-dir ./my-plugin appears to be a significant change in how local plugins are installed. Verify that this new command syntax is correct and that the --plugin-dir flag is the proper way to load local plugins in the current version of Claude Code. The old command suggested installing from a local directory, while the new approach suggests starting Claude with a plugin directory parameter.

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.

2 participants