Skip to content

Add data guideline for Safari backported releases#11909

Merged
ddbeck merged 6 commits intomdn:mainfrom
queengooborg:docs/safari-backported-releases
Oct 18, 2021
Merged

Add data guideline for Safari backported releases#11909
ddbeck merged 6 commits intomdn:mainfrom
queengooborg:docs/safari-backported-releases

Conversation

@queengooborg
Copy link
Contributor

This PR is an attempt to add a guideline for the Safari backport releases (4.1, 6.1, 6.2, and 7.1) based upon the discussions in #9423. This removes our initial guideline where we state the version number should be for the backported release, adding a section just below it to document these special Safari releases and what to do to deal with them.

Note: I'm writing this section under the assumption that we will remove ALL backported releases from Safari, though we haven't actually come to a conclusion quite yet. I'm happy to change this PR as needed!

@github-actions github-actions bot added the docs Issues or pull requests regarding the documentation of this project. label Aug 4, 2021
Copy link
Contributor

@ddbeck ddbeck left a comment

Choose a reason for hiding this comment

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

A couple ideas for you, though subject to change depending on the outcome of the issue discussion.

@ddbeck
Copy link
Contributor

ddbeck commented Aug 25, 2021

@vinyldarkscratch hey Vinyl, on the most recent BCD call we talked about this. It sounds like this is ready to re-visit (with the overall decision reflected by the text, my quibbles notwithstanding).

@foolip
Copy link
Contributor

foolip commented Aug 26, 2021

Because of #11755 I've noticed there's another case to consider here: Safari 1.3. It was released 2 weeks before Safari 2 and Wikipedia says "Released with Mac OS X v10.3.9. Included most of the rendering speed and website compatibility improvements that were developed for 2.0. designMode support."

@vinyldarkscratch do you have access to both 1.3 and 2 in order to test for differences between the two?

@queengooborg
Copy link
Contributor Author

@vinyldarkscratch do you have access to both 1.3 and 2 in order to test for differences between the two?

I do have a Safari 1.3 VM, but unfortunately I don't have access to it at the moment while my laptop is undergoing repairs. When I get my laptop back, though, I'll be sure to check them out! (Sadly checking differences will be harder since the collector isn't supported below Safari 3.)

@foolip
Copy link
Contributor

foolip commented Aug 30, 2021

I see, hope your laptops comes back strong and healthy from the repairs :)

This isn't really blocking this issue, we can write a guideline and then see after the fact whether Safari 1.3 should be removed or not based on that guideline.

@queengooborg queengooborg marked this pull request as draft September 4, 2021 10:17
@queengooborg
Copy link
Contributor Author

Marking this as a draft for the time being! I'll be revising these docs based upon your review, @ddbeck, and will mark this as ready once I do!

@queengooborg queengooborg marked this pull request as ready for review September 16, 2021 06:42
Copy link
Contributor

@ddbeck ddbeck left a comment

Choose a reason for hiding this comment

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

I think we're getting close! Some suggestions in line comments.

queengooborg and others added 2 commits October 15, 2021 09:33
Co-authored-by: Daniel D. Beck <daniel@ddbeck.com>
Copy link
Contributor

@ddbeck ddbeck left a comment

Choose a reason for hiding this comment

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

Thank you! 🎉

@ddbeck ddbeck merged commit 6c8acde into mdn:main Oct 18, 2021
@queengooborg queengooborg deleted the docs/safari-backported-releases branch October 18, 2021 17:32
ddbeck added a commit to ddbeck/browser-compat-data that referenced this pull request Oct 19, 2021
ddbeck added a commit that referenced this pull request Oct 19, 2021
* Bump version to v4.0.7

* Set release date

* Add release note for #12824

* Add release note for #11909

* Add release note for #12818

* Add release note for #12832

* Add release note for #12836

* Hoist known issues

* Add stats

* Add link to new data guideline
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Issues or pull requests regarding the documentation of this project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants