Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • autosubmit autosubmit
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 338
    • Issues 338
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 21
    • Merge requests 21
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • Earth SciencesEarth Sciences
  • autosubmitautosubmit
  • Merge requests
  • !435

Add dependency groups (optionals) to our setup.py

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Bruno de Paula Kinoshita requested to merge group-deps-pytest into master May 16, 2024
  • Overview 36
  • Commits 1
  • Pipelines 0
  • Changes 1

Part of #1285 and !426

@dbeltran, with PostgreSQL support, we will bring more dependencies to the project. However, for sites where PostgreSQL is not used, it would not make sense to have both sqlite and postgres. Likewise, for those using PostgreSQL, installing sqlite shouldn't be a requirement.

This MR introduces the extras/groups options to our setup.py file.

The test dependencies are also being made optional. This way, the ClimateDT VM, Fugaku, Leonardo, LUMI, etc, won't have pytest or mock installed, as those are not runtime requirements.

And as part of #961, this MR is replacing the command used to run the tests on the workers. This way, we can add pytest tests to the PostgreSQL MR, and pytest will run both unittest & pytest tests, and we can slowly migrate our tests, superseding !391 .

This is part of the work @ltenorio started on his MR, but I think we can merge it before that one.

Edited Aug 20, 2024 by Bruno de Paula Kinoshita
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: group-deps-pytest