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

refactor: remove tox py3X tests SC-1598 #2684

Merged
merged 1 commit into from
Aug 9, 2023
Merged

Conversation

CalvoM
Copy link
Contributor

@CalvoM CalvoM commented Aug 1, 2023

Why is this needed?

Removing the py3X tests in tox.ini since we are moving testing to building in the specified
series.

Test Steps

Checklist

  • I have updated or added any unit tests accordingly
  • I have updated or added any integration tests accordingly
  • Changes here need to be documented, and this was done in:

Does this PR require extra reviews?

  • Yes
  • No

@github-actions
Copy link

github-actions bot commented Aug 1, 2023

Jira: SC-1598

GitHub Issues: No GitHub issues are fixed by this PR. (No commits have Fixes: #### references)

Launchpad Bugs: No Launchpad bugs are fixed by this PR. (No commits have LP: #### references)

Documentation: The changes in this PR do not require documentation changes.

👍 this comment to confirm that this is correct.

@renanrodrigo
Copy link
Member

You could also remove tools/setup_pyenv.sh as we don't really need pyenv anymore

tox.ini Show resolved Hide resolved
@CalvoM CalvoM force-pushed the refactor-tox-testing branch 2 times, most recently from 90d346e to b3915ac Compare August 3, 2023 17:18
Copy link
Member

@renanrodrigo renanrodrigo left a comment

Choose a reason for hiding this comment

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

Great, thanks much!

@renanrodrigo renanrodrigo merged commit b767d63 into main Aug 9, 2023
17 of 23 checks passed
@renanrodrigo renanrodrigo deleted the refactor-tox-testing branch August 9, 2023 20:54
@CalvoM CalvoM changed the title refactor: remove tox py3X tests refactor: remove tox py3X tests SC-1598 Aug 17, 2023
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.

3 participants