Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🚮 Sweep experiments older than 2020-02-29 #2

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Mar 1, 2021

Sweep experiments last flipped globally up to 2020-02-29:

  • (2019-10-01, 0ab715e) amp-action-macro: 1
  • (2018-11-05, d26327a) amp-story-responsive-units: 1
  • (2018-07-11, 366b9d5) amp-story-v1: 1
  • (2018-07-11, 366b9d5) chunked-amp: 1
  • (2019-10-28, 4ca1409) fix-inconsistent-responsive-height-selection: 0
  • (2017-05-16, ca20059) pump-early-frame: 1

⚠️ Javascript source files require intervention

The following may contain errors and/or require intervention to remove superfluous conditionals:

  • extensions/amp-action-macro/0.1/amp-action-macro.js
  • extensions/amp-action-macro/0.1/test/test-amp-action-macro.js
  • extensions/amp-story/1.0/test/test-amp-story.js
  • src/runtime.js
  • test/unit/test-runtime.js

Refer to the removal guide for suggestions on handling these modified Javascript files.


⚠️ HTML files may still contain references

The following HTML files contain references to experiment names which may be stale and should be manually removed:

  • examples/amp-action-macro.html

Refer to the removal guide for suggestions on handling these HTML files.

Previous history on prod-config.json:

- 0ab715e - 2019-10-01T10:32:36-04:00 - Launch <amp-action-macro>
Previous history on prod-config.json:

- d26327a - 2018-11-05T12:02:25-05:00 - Re-launch amp-story-responsive-units experiment
- d5ec88b - 2018-10-30T16:01:59-07:00 - Unlaunch amp-story-responsive-units for urgent release
- 65379df - 2018-10-26T15:27:09-04:00 - Launch amp-story-responsive-units experiment
Previous history on prod-config.json:

- 366b9d5 - 2018-07-11T12:03:09-04:00 - Revert "Remove launched experiments. (ampproject#16528)"
- 3a3dbd9 - 2018-07-06T14:47:49-04:00 - Remove launched experiments.
- 1194c16 - 2018-06-12T11:38:38-04:00 - Start canonical adsense fast fetch experiments at 1%
Previous history on prod-config.json:

- 366b9d5 - 2018-07-11T12:03:09-04:00 - Revert "Remove launched experiments. (ampproject#16528)"
- 3a3dbd9 - 2018-07-06T14:47:49-04:00 - Remove launched experiments.
- 34ffcdb - 2017-02-17T13:10:59-08:00 - turn on chunked-amp
Previous history on prod-config.json:

- 4ca1409 - 2019-10-28T15:54:36+00:00 - Ramp down inconsistent height selection fix experiment to 0%. This doesn't work very well on desktop / tablet so we need to fix that before proceeding.
Previous history on prod-config.json:

- ca20059 - 2017-05-16T09:31:30-04:00 - Start Doubleclick Fast Fetch potential SRA delay measure experiment
Sweep experiments last flipped globally up to 2020-02-29:

- (2019-10-01, 0ab715e) `amp-action-macro`: 1
- (2018-11-05, d26327a) `amp-story-responsive-units`: 1
- (2018-07-11, 366b9d5) `amp-story-v1`: 1
- (2018-07-11, 366b9d5) `chunked-amp`: 1
- (2019-10-28, 4ca1409) `fix-inconsistent-responsive-height-selection`: 0
- (2017-05-16, ca20059) `pump-early-frame`: 1

---

### ⚠️ Javascript source files require intervention

The following may contain errors and/or require intervention to remove superfluous conditionals:

- `extensions/amp-action-macro/0.1/amp-action-macro.js`
- `extensions/amp-action-macro/0.1/test/test-amp-action-macro.js`
- `extensions/amp-story/1.0/test/test-amp-story.js`
- `src/runtime.js`
- `test/unit/test-runtime.js`

Refer to the removal guide for [suggestions on handling these modified Javascript files.](https://github.com/ampproject/amphtml/blob/master/build-system/tasks/sweep-experiments/README.md#followup)

---

### ⚠️ HTML files may still contain references

The following HTML files contain references to experiment names which may be stale and should be manually removed:

- `examples/amp-action-macro.html`

Refer to the removal guide for [suggestions on handling these HTML files.](https://github.com/ampproject/amphtml/blob/master/build-system/tasks/sweep-experiments/README.md#followup:html)
@github-actions
Copy link
Author

github-actions bot commented Mar 1, 2021

You may checkout this pull request to follow-up manually:

git checkout -t upstream/sweep-experiments-339af2400

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