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.

Sign up for GitHub

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

Jump to bottom

API stability testing #23

Closed
richtja opened this issue Sep 24, 2024 · 1 comment
Closed

API stability testing #23

richtja opened this issue Sep 24, 2024 · 1 comment
Assignees
@harvey0100

Comments

@richtja
Copy link
Contributor

richtja commented Sep 24, 2024 •
edited
Loading

Is your feature request related to a problem? Please describe.
We need to ensure that autils API is stable over different versions. Since the autils will be used through different projects, we must avoid changes to the API of utilities until it is really necessary. This proposal is about possibilities how to ensure that.

Describe the solution you'd like
A clear and concise description of what you want to happen.
Add PR and release testing which will run the patch or new version against the tests from master of older version to ensure there is no regression in the API.

Additional information
Add any additional information or screenshots about the feature request here.

If the API change is necessary we need to come up with suitable deprecating process for everybody to have time to adapt to such change.

Part of #1
@mr-avocado mr-avocado bot moved this to Triage in Default project Sep 24, 2024
@richtja richtja assigned richtja and harvey0100 and unassigned richtja Oct 9, 2024
harvey0100 added a commit to harvey0100/autils that referenced this issue Oct 18, 2024
@harvey0100
API Stability
54246dc 
Autils code is fetched from the pull request using the github.event.pull_request.head.sha.
The tests are fetched from the master branch by running git fetch origin master
and checking out only the tests/ directory from the master branch.

Reference:      avocado-framework#23
Signed-off-by: Harvey Lynden <hlynden@redhat.com>
@harvey0100 harvey0100 mentioned this issue Oct 18, 2024
Open
harvey0100 added a commit to harvey0100/autils that referenced this issue Oct 18, 2024
@harvey0100
API Stability
83f6791 
Autils code is fetched from the pull request using the github.event.pull_request.head.sha.
The tests are fetched from the master branch by running git fetch origin master
and checking out only the tests/ directory from the master branch.

Reference:      avocado-framework#23
Signed-off-by: Harvey Lynden <hlynden@redhat.com>
@harvey0100 harvey0100 mentioned this issue Oct 18, 2024
Merged
harvey0100 added a commit to harvey0100/autils that referenced this issue Nov 1, 2024
@harvey0100
API Stability
9659d07 
Autils code is fetched from the pull request using the github.event.pull_request.head.sha.
The tests are fetched from the master branch by running git fetch origin master
and checking out only the tests/ directory from the master branch.

Reference:      avocado-framework#23
Signed-off-by: Harvey Lynden <hlynden@redhat.com>
@richtja
Copy link
Contributor Author

richtja commented Nov 4, 2024

Solved in #25

@richtja richtja closed this as completed Nov 4, 2024
@github-project-automation github-project-automation bot moved this from Short Term (Current Q) Backlog to Done 109 in Default project Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@harvey0100 harvey0100

Labels
None yet
Projects
Default project
Status: Done 109
Milestone
No milestone
Development

No branches or pull requests
2 participants
@richtja @harvey0100