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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃毊 Sweep experiments older than 2020-01-01 #1

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 Jan 1, 2021

Sweep experiments last flipped globally up to 2020-01-01:

  • (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-01-01:

- (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 Jan 1, 2021

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

git checkout -t upstream/sweep-experiments-799565daa

@rmtnong
Copy link

rmtnong commented Apr 5, 2021

Gui

@jkoooo4456
Copy link

Tina

@@ -1,23 +1,17 @@
{
"allow-doc-opt-in": ["amp-next-page", "analytics-chunks-inabox"],
"allow-url-opt-in": ["pump-early-frame"],

Choose a reason for hiding this comment

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

J

@ZOHAN1102
Copy link

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

git checkout -t upstream/sweep-experiments-799565daa

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants