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

chore(deps): remove future #1353

Closed
wants to merge 1 commit into from
Closed

chore(deps): remove future #1353

wants to merge 1 commit into from

Conversation

Nytelife26
Copy link
Member

This relates to...

Issue #1352

Changes

  • Removes the future dependency (redundant)

Features

N/A

Bug Fixes

N/A

Breaking Changes and Deprecations

N/A

@Nytelife26 Nytelife26 requested a review from suchow as a code owner August 7, 2023 02:08
@Nytelife26 Nytelife26 added type: chore Issues and PRs related to changes that do not fit into other categories. priority: low Issues and PRs that should be resolved, but can be postponed. version: patch Issues and PRs with bug fixes belonging to the next patch release. status: review-ready PRs that are ready for author review. cat: maintenance Issues and PRs related to the maintenance of a module. labels Aug 7, 2023
Nytelife26 added a commit that referenced this pull request Aug 31, 2023
@Nytelife26
chore: remove future
690f8c0 
this removes the need for #1353 but it was necessary to clear setuptools legacy
Nytelife26 added a commit that referenced this pull request Sep 9, 2023
@Nytelife26
chore: update test suite and dependencies
4cbeaf0 
note the removal of futures in this commit.
this removes the need for #1353 but it was necessary to clear setuptools legacy.
Nytelife26 added a commit that referenced this pull request Jan 24, 2024
@Nytelife26
ci: fix test suite (#1354)
629af47 
* chore: raise min python to 3.6.7

* ci(lint-test): update actions

* chore: lint

* test: fix for windows

path resolution was not padding slashes enough on windows in
illegal-chars

* chore: update test suite and dependencies

note the removal of futures in this commit.
this removes the need for #1353 but it was necessary to clear setuptools legacy.
@Nytelife26 Nytelife26 closed this Jan 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@suchow suchow Awaiting requested review from suchow suchow is a code owner

Assignees

@suchow suchow

@Nytelife26 Nytelife26

Labels
cat: maintenance Issues and PRs related to the maintenance of a module. priority: low Issues and PRs that should be resolved, but can be postponed. status: review-ready PRs that are ready for author review. type: chore Issues and PRs related to changes that do not fit into other categories. version: patch Issues and PRs with bug fixes belonging to the next patch release.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Nytelife26 @suchow