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

Add API Changelog #4612

Merged
merged 15 commits into from
Jul 19, 2023
Merged

Add API Changelog #4612

merged 15 commits into from
Jul 19, 2023

Conversation

intelliot
Copy link
Collaborator

High Level Overview of Change

Introduce an API Changelog, which logs the changes that have been made to the API.

Context of Change

It is currently difficult to keep track of the changes and additions that are made to the rippled API. While all changes are surfaced in PRs and Release Notes, these are mixed in with other non-API-affecting changes. PRs that affect the API have the API Change label applied, but it is hard to identify which PRs have been included in each release. Furthermore, some API changes will take effect based on api_version (starting with rippled version 1.12, which will introduce api_version: 2), while others are based on the version of rippled.

The API Changelog clarifies the details of the changes in a way that is easily understood by API consumers, and breaks down the changes so it's clear which ones are gated by api_version (while others are determined by rippled version).

Type of Change

  • Documentation Updates

Future Tasks

From now on, all PR authors are responsible for updating the API Changelog according to the additions/changes that their PR makes to the APIs.

@intelliot intelliot merged commit aded4a7 into XRPLF:develop Jul 19, 2023
15 checks passed
@intelliot intelliot deleted the api-changelog branch July 19, 2023 21:03
@intelliot intelliot added this to the 1.12 milestone Jul 19, 2023
This was referenced Aug 18, 2023
ckeshava pushed a commit to ckeshava/rippled that referenced this pull request Sep 22, 2023
Introduce an API Changelog, which logs the changes that have been made
to the API.

Without this changelog, it is difficult to keep track of the changes and
additions that are made to the API. While all changes are surfaced in
PRs and Release Notes, these are mixed in with other non-API-affecting
changes. PRs that affect the API have the `API Change` label applied,
but it is hard to identify which PRs have been included in each release.
Furthermore, some API changes will take effect based on `api_version`
(starting with rippled version 1.12, which will introduce `api_version:
2`), while others are based on the `rippled` version.

The API Changelog clarifies the details of the changes in a way that is
easily understood by API consumers, and breaks down the changes to be
clear which ones are gated by api_version (versus `rippled` version).

From now on, all PR authors are responsible for updating the API
Changelog according to the additions/changes that their PR makes to the
APIs.
ckeshava pushed a commit to ckeshava/rippled that referenced this pull request Sep 25, 2023
Introduce an API Changelog, which logs the changes that have been made
to the API.

Without this changelog, it is difficult to keep track of the changes and
additions that are made to the API. While all changes are surfaced in
PRs and Release Notes, these are mixed in with other non-API-affecting
changes. PRs that affect the API have the `API Change` label applied,
but it is hard to identify which PRs have been included in each release.
Furthermore, some API changes will take effect based on `api_version`
(starting with rippled version 1.12, which will introduce `api_version:
2`), while others are based on the `rippled` version.

The API Changelog clarifies the details of the changes in a way that is
easily understood by API consumers, and breaks down the changes to be
clear which ones are gated by api_version (versus `rippled` version).

From now on, all PR authors are responsible for updating the API
Changelog according to the additions/changes that their PR makes to the
APIs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants