Skip to content

summary name, fix markdown error#2304

Merged
WilcoFiers merged 2 commits intodevelopfrom
WilcoFiers-patch-5
Apr 10, 2025
Merged

summary name, fix markdown error#2304
WilcoFiers merged 2 commits intodevelopfrom
WilcoFiers-patch-5

Conversation

@WilcoFiers
Copy link
Copy Markdown
Member

There is no "explicit semantic role" link. Should be "explicit role" as seen in other places in the rule.

Need for Call for Review: none, editorial


How to Review And Approve

  • Go to the “Files changed” tab
  • Here you will have the option to leave comments on different lines.
  • Once the review is completed, find the “Review changes” button in the top right, select “Approve” (if you are really confident in the rule) or "Request changes" and click “Submit review”.
  • Make sure to also review the proposed Call for Review period. In case of disagreement, the longer period wins.

Copy link
Copy Markdown
Collaborator

@Jym77 Jym77 left a comment

Choose a reason for hiding this comment

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

Good. Not approving since Dan already did.

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 10, 2025

Deploy Preview for act-rules ready!

Name Link
🔨 Latest commit b4b169f
🔍 Latest deploy log https://app.netlify.com/sites/act-rules/deploys/67f7c028ca391b0008bfd688
😎 Deploy Preview https://deploy-preview-2304--act-rules.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@WilcoFiers WilcoFiers merged commit 30b5343 into develop Apr 10, 2025
6 checks passed
@WilcoFiers WilcoFiers deleted the WilcoFiers-patch-5 branch April 10, 2025 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants