Skip to content

Disable individual static rules in DNR#33733

Merged
rebloor merged 7 commits intomdn:mainfrom
rebloor:Disable-individual-static-rules-in-DNR
Jun 12, 2024
Merged

Disable individual static rules in DNR#33733
rebloor merged 7 commits intomdn:mainfrom
rebloor:Disable-individual-static-rules-in-DNR

Conversation

@rebloor
Copy link
Copy Markdown
Contributor

@rebloor rebloor commented May 23, 2024

Description

Provides documentation for Bug 1810762 [DNR] Disable individual static rules, including:

  • release notes
  • documentation of getDisabledRuleIds and updateStaticRules

Related issues and pull requests

BCD changes in mdn/browser-compat-data#23168

@rebloor rebloor added the Content:WebExt WebExtensions docs label May 23, 2024
@rebloor rebloor requested a review from rpl May 23, 2024 19:17
@rebloor rebloor self-assigned this May 23, 2024
@rebloor rebloor requested review from a team as code owners May 23, 2024 19:17
@rebloor rebloor requested review from hamishwillee and removed request for a team May 23, 2024 19:17
@github-actions github-actions Bot added Content:Firefox Content in the Mozilla/Firefox subtree size/m [PR only] 51-500 LoC changed labels May 23, 2024
Comment thread files/en-us/mozilla/firefox/releases/128/index.md Outdated
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 23, 2024

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Copy Markdown
Member

@rpl rpl left a comment

Choose a reason for hiding this comment

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

@rebloor thanks! follows an initial review pass (which include a couple of request for changes, a couple of nits and some thoughts and questions).

Comment on lines +32 to +34
## Examples

{{WebExtExamples}}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Question: do we have an example snippet that is meant to be interpolated here? or should we omit this section in this PR and introduce the section back in a separate PR if we want to followup with a code snippet example?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

None that I'm aware of.

Comment on lines +32 to +34
## Examples

{{WebExtExamples}}
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

None that I'm aware of.

@rebloor rebloor requested a review from rpl May 28, 2024 04:25
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions github-actions Bot added the merge conflicts 🚧 [PR only] label Jun 4, 2024
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 4, 2024

This pull request has merge conflicts that must be resolved before it can be merged.

@hamishwillee hamishwillee removed their request for review June 4, 2024 07:23
@github-actions github-actions Bot removed the merge conflicts 🚧 [PR only] label Jun 4, 2024
@rebloor rebloor requested a review from rpl June 5, 2024 10:48
Copy link
Copy Markdown
Member

@rpl rpl left a comment

Choose a reason for hiding this comment

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

@rebloor thanks a lot, apologies for not having been able to come back to this PR last week, this PR looks good and there are only two tweaks to the wording we discussed that seems to need to be applied before merging it.

r+wc, where "with comments" part referring to applying the tweaks to the wording from https://github.com/mdn/content/pull/33733/files#r1627480918 and https://github.com/mdn/content/pull/33733/files#r1616558474

@rebloor rebloor merged commit 3448670 into mdn:main Jun 12, 2024
@rebloor rebloor deleted the Disable-individual-static-rules-in-DNR branch June 12, 2024 03:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:Firefox Content in the Mozilla/Firefox subtree Content:WebExt WebExtensions docs size/m [PR only] 51-500 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants