Skip to content

Transition to pure pyproject.toml for project metadata - #351

Merged
gsheni merged 6 commits into
mainfrom
pyprojecttoml
Nov 23, 2022
Merged

Transition to pure pyproject.toml for project metadata #351
gsheni merged 6 commits into
mainfrom
pyprojecttoml

Conversation

@gsheni

@gsheni gsheni commented Nov 22, 2022

Copy link
Copy Markdown
Contributor

No description provided.

@gsheni gsheni self-assigned this Nov 22, 2022
@codecov

codecov Bot commented Nov 22, 2022

Copy link
Copy Markdown

Codecov Report

Merging #351 (8fb6b49) into main (a844f05) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #351   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           29        28    -1     
  Lines         1341      1335    -6     
=========================================
- Hits          1341      1335    -6     
Impacted Files Coverage Δ
composeml/data_slice/extension.py 100.00% <0.00%> (ø)
composeml/__init__.py

@gsheni gsheni changed the title Transition to pure pyproject.toml Transition to pure pyproject.toml for project metadata Nov 22, 2022
@gsheni
gsheni marked this pull request as ready for review November 22, 2022 22:40
Comment thread docs/source/release_notes.rst Outdated
Future Release
==============
* Enhancements
* Transition to pure pyproject.toml for project metadata (:pr:`351`)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Enhancement or change?

matrix:
os: [ubuntu-latest, macos-latest]
python_version: ["3.8", "3.9", "3.10"]
python_version: ["3.8", "3.9", "3.10", "3.11"]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If we are going to support 3.11, shouldn't we also run all our unit tests with 3.11, not just the install test?

Comment thread pyproject.toml Outdated
"scikit-learn >= 0.20.0, !=0.22",
]
complete = [
"featuretools[updater]",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

featuretools -> compose

Comment thread pyproject.toml
docs = [
"evalml >= 0.45.0"
]
dev = [

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Previously the dev requirements also included the test requirements. Should we also add compose[test] in this section?

@gsheni
gsheni merged commit 62b39bd into main Nov 23, 2022
@gsheni
gsheni deleted the pyprojecttoml branch November 23, 2022 17:59
@gsheni gsheni mentioned this pull request Jan 6, 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