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

Bump the API version #319

Merged
merged 2 commits into from May 14, 2018
Merged

Bump the API version #319

merged 2 commits into from May 14, 2018

Conversation

algernon
Copy link
Contributor

Due to the plugin API redesign, plugins that migrate may want to ensure they are compiled against a recent enough Kaleidoscope. Others may opt to provide separate implementations for each version. For this to work, we need to bump the API version.

@obra
Copy link
Member

obra commented May 13, 2018

should this get an upgrading.md entry?

@algernon
Copy link
Contributor Author

Indeed, it should. Thanks for the reminder! Will add that soon(ish).

@obra
Copy link
Member

obra commented May 13, 2018 via email

Due to the plugin API redesign, plugins that migrate may want to ensure they are
compiled against a recent enough Kaleidoscope. Others may opt to provide
separate implementations for each version. For this to work, we need to bump the
API version.

Signed-off-by: Gergely Nagy <algernon@keyboard.io>
Rearranged a little, so deprecation is a section, and we can put other things,
notes, into the document. With the rearrangement, added a section about the new
plugin API, and explained the API version bump too.

Signed-off-by: Gergely Nagy <algernon@keyboard.io>
@algernon algernon merged commit d5251e8 into master May 14, 2018
@algernon algernon deleted the f/api-bump branch May 14, 2018 05:02
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.

None yet

2 participants