Skip to content

dev to main#494

Merged
jth-nw merged 5 commits intomainfrom
dev
Mar 12, 2026
Merged

dev to main#494
jth-nw merged 5 commits intomainfrom
dev

Conversation

@jth-nw
Copy link
Collaborator

@jth-nw jth-nw commented Mar 12, 2026

No description provided.

hilram7 and others added 5 commits March 11, 2026 18:06
Changed relative path to /docs/kb/ format so the KB copy script can
properly rewrite links when copying to versioned directories.

The link to determining-the-number-of-enabled-microsoft-entra-id-accounts
with #instructions anchor was using relative path ../../ which broke when
copied to versions 10.6, 10.7, and 10.8.

The #instructions anchor exists on the target page (line 33).
* Move the firewall policy to the service policy

* Remove extra links

* Fix issues

* Fix issues
Run Dale in a separate claude-code-action step with its own turn budget,
write results to JSON, then post as inline comments via a shell step.
Editorial review updated to reference Dale count and avoid duplicating
linter findings. Followup job resolves and dismisses both Vale and Dale
threads after fixes.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
feat: add Dale linting as inline PR review comments
@jth-nw jth-nw requested review from a team as code owners March 12, 2026 16:09
@jth-nw jth-nw merged commit 8cdbeb2 into main Mar 12, 2026
5 of 7 checks passed
- name: Run Dale linting
id: dale
if: steps.changed-files.outputs.count > 0
uses: anthropics/claude-code-action@v1

Check warning

Code scanning / CodeQL

Unpinned tag for a non-immutable Action in workflow

Unpinned 3rd party Action 'Doc PR Review' step [Uses Step: dale](1) uses 'anthropics/claude-code-action' with ref 'v1', not a pinned commit hash
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.

4 participants