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

feat: update package config #1448

Open
wants to merge 15 commits into
base: develop
Choose a base branch
from

Conversation

vorel99
Copy link
Contributor

@vorel99 vorel99 commented Sep 17, 2023

In python world, new standard for package configuration is pyproject.toml. So I moved all static config from setup.py to pyproject.toml.

  • move all static package setup from setup.py to pyproject.toml
  • add commands to Makefile
    • install_dev: install all requirements for pandas development and testing
    • install_test_pandas: install ydata-profiling with optional dependencies for testing
    • install_test_spark: install ydata-profiling with optional dependencies for testing spark
    • remove all requirements files
  • update spark testing
    • delete pytest-spark
    • add spark_session as pytest fixture
  • update gitlab actions
    • replace all requirements files usage
    • remove pyspark tests with version of pandas 0.25.3

@codecov-commenter
Copy link

codecov-commenter commented Sep 17, 2023

Codecov Report

Patch coverage has no change and project coverage change: +0.04% 🎉

Comparison is base (fdc0346) 89.49% compared to head (8632412) 89.53%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1448      +/-   ##
===========================================
+ Coverage    89.49%   89.53%   +0.04%     
===========================================
  Files          194      194              
  Lines         6318     6316       -2     
===========================================
+ Hits          5654     5655       +1     
+ Misses         664      661       -3     
Flag Coverage Δ
py3.8-ubuntu-22.04-pandas 89.53% <0.00%> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
setup.py 0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@fabclmnt
Copy link
Contributor

Hi @vorel99 ,

once again than you for your contribution :)

I've already requested the review from the team!

@vorel99 vorel99 changed the title refactor: update package config feat: update package config Sep 19, 2023
* Update javascript.html

* Update style.html
.gitignore Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
requirements-dev.txt Outdated Show resolved Hide resolved
requirements-spark.txt Outdated Show resolved Hide resolved
requirements-test.txt Outdated Show resolved Hide resolved
requirements.txt Outdated Show resolved Hide resolved
@vascoalramos
Copy link
Contributor

vascoalramos commented Sep 20, 2023

Hi @vorel99.

Thank you for your work in the migration to pyproject.toml.
I left some change requests mostly related to the makefile and dependency options.
The primary reason for most of the requests relates to keeping consistency between all of our open-source projects.

@vorel99 vorel99 marked this pull request as ready for review September 20, 2023 17:30
Copy link
Contributor

@vascoalramos vascoalramos left a comment

Choose a reason for hiding this comment

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

Everything else seems fine to me. @portellaa, what do you think?

Makefile Outdated Show resolved Hide resolved
@vorel99
Copy link
Contributor Author

vorel99 commented Oct 9, 2023

hello,
@vascoalramos @portellaa what is the status of MR? Is there any blocker to be resolved?

@aquemy aquemy force-pushed the develop branch 2 times, most recently from 6a3342a to 8f4f622 Compare October 10, 2023 10:25
@portellaa
Copy link
Member

Hi @vorel99

Thanks for your PR.

Unfortunately we don't have the time right now to move forward with this, we kindly ask you to give us some more time.
Given the amount of the changes, it breaks some internal alignments that we will need to work with you, but right now we can't focus on this.

Sorry for this.
We will get in touch as soon as we can pick this.

Cheers

@vorel99
Copy link
Contributor Author

vorel99 commented Oct 11, 2023

Thanks for reply @portellaa,
there wasn't that many of changes (its probably result of force-push from aquemy).
Changes were just in 15 files with 189 additions and 171 deletions.
There is an example of pull request, what it looked like when it was created and vascoalramos did review.
vorel99#78

I can rebase my branch to new develop, when you are ready.

Regards,
JC

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.

None yet

9 participants