Skip to content

Stop using the instructional flag. - #7

Merged
Hyperkid123 merged 1 commit into
RedHatInsights:masterfrom
Hyperkid123:stop-using-instructiona
Nov 21, 2022
Merged

Stop using the instructional flag.#7
Hyperkid123 merged 1 commit into
RedHatInsights:masterfrom
Hyperkid123:stop-using-instructiona

Conversation

@Hyperkid123

Copy link
Copy Markdown
Contributor

No description provided.

@Hyperkid123
Hyperkid123 merged commit c209e39 into RedHatInsights:master Nov 21, 2022
@Hyperkid123
Hyperkid123 deleted the stop-using-instructiona branch November 21, 2022 10:06
platex-rehor-bot added a commit to platex-rehor-bot/learning-resources that referenced this pull request Apr 16, 2026
RHCLOUD-46729
Reword convention RedHatInsights#7 to make clear that useChrome is the
sole public interface — don't import other Chrome internals.
Addresses coderabbit review nitpick.
karelhala pushed a commit that referenced this pull request Apr 16, 2026
* docs: add agent readiness files

RHCLOUD-46729
Add AGENTS.md with project structure, conventions, module
federation table, tech stack, pitfalls, and doc index.
Add docs/*-guidelines.md for testing, security, error
handling, and integration. Prepend @AGENTS.md import and
build commands to existing CLAUDE.md.

* docs: add application parts section

RHCLOUD-46729
Address reviewer feedback: add Application Parts section to AGENTS.md
describing the 3 user-facing surfaces (all learning resources page,
bundle-specific pages, help panel) plus the creator tool. Add missing
npm run verify command to claude.md.

* docs: document API call exceptions

RHCLOUD-46729
Update convention #9 to acknowledge 5 legacy components
with inline API calls. Mark as tech debt to migrate when
touching those files.

* docs: clarify Chrome API abstraction

RHCLOUD-46729
Reword convention #7 to make clear that useChrome is the
sole public interface — don't import other Chrome internals.
Addresses coderabbit review nitpick.
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