Skip to content

Update Addon API min ver#11912

Merged
feerrenrut merged 4 commits intobetafrom
addonApiBreak2021.1
May 21, 2021
Merged

Update Addon API min ver#11912
feerrenrut merged 4 commits intobetafrom
addonApiBreak2021.1

Conversation

@feerrenrut
Copy link
Copy Markdown
Contributor

2021.1 will have several API breaking changes.
Addons need to be re-tested and have their manifest updated.

This does not need to be merged until 2021.1 branches off for beta. Doing so earlier would cause add-ons to be marked as incompatible for alpha users. However by the time the beta comes, addon authors should have been able to test and provide pre-release versions of their addons.

2021.1 will have several API breaking changes.
Addons need to be re-tested and have their manifest updated.
@feerrenrut feerrenrut added this to the 2021.1 milestone Dec 7, 2020
@feerrenrut feerrenrut self-assigned this Mar 22, 2021
@seanbudd seanbudd mentioned this pull request May 10, 2021
@feerrenrut feerrenrut changed the base branch from master to beta May 17, 2021 06:24
@feerrenrut feerrenrut requested a review from a team as a code owner May 17, 2021 06:24
@feerrenrut feerrenrut requested review from seanbudd and removed request for a team May 17, 2021 06:24
seanbudd
seanbudd previously approved these changes May 17, 2021
@LeonarddeR
Copy link
Copy Markdown
Collaborator

Given there are more and more add-ons breaking, I'd like to propose to merge this ASAP. See also #12426

@feerrenrut
Copy link
Copy Markdown
Contributor Author

We intentionally haven't merged this because:

  • Merging this signifies the commitment to the API.
  • Once we merge this, all addon's must update their manifest before they can even test for compatibility. Delaying this process allows authors and adventurous addon users to test and report problems with the upcoming API without a new addon release being made.
  • The downside is that users are not warned that add-ons may not work. However remember, this is alpha software, there will be rough edges.

@josephsl
Copy link
Copy Markdown
Contributor

josephsl commented May 19, 2021 via email

Copy link
Copy Markdown
Collaborator

@LeonarddeR LeonarddeR left a comment

Choose a reason for hiding this comment

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

Looks good to me now.

@LeonarddeR
Copy link
Copy Markdown
Collaborator

We intentionally haven't merged this because:

Makes sense. Having said that, i think this might yet be the time to go with this change:

  1. Alpha started with the 2021.2 release cycle, so I assume the API of 2021.1 can be considered stable?
  2. It will urge add-on authors once more to fix their add-ons.

@feerrenrut
Copy link
Copy Markdown
Contributor Author

I believe we are very close to doing this.

Alpha started with the 2021.2 release cycle, so I assume the API of 2021.1 can be considered stable?

Note quite, we allowed 2021.2 to start so that changes intended specifically for that release could be merged without effecting 2021.1.

It will urge add-on authors once more to fix their add-ons.

Yes, that would be good.

@feerrenrut feerrenrut merged commit 6852a8a into beta May 21, 2021
@feerrenrut feerrenrut deleted the addonApiBreak2021.1 branch May 21, 2021 05:29
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.

many addons have been disabled after updating to snapshot Version: alpha-22782,7e6561a0 U

4 participants